[PATCH] fixes of tcp_hostcache.c
Andre Oppermann
andre at freebsd.org
Tue Dec 2 05:06:15 PST 2003
> Greetings,
>
> I found two calls of bcopy() in tcp_hostcache.c got the source and the
> destination pointer wrongly.
> # Perhaps the author got confused with memcpy(), I guess.
Indeed. Originally these were memcpy calls and I've been asked to change
them to bcopy calls. Which I did. It didn't occur to me that bcopy would
change the destination from left-hand to right-hand argument... But that
seems to be a common pitfall. :/
> The fix is attached as a patch against tcp_hostcache.c as of revision 1.2.
>
> Please review and incorporate the patch to the tree because the bug makes
> hostcache useless with TCPv6 and/or brings TTCP (RFC1644) meaningless.
> Especially the bug renders it unusable to use TCPv6 and TTCP together.
> # To confirm this: sysctl net.inet.tcp.rfc1644=1; telnet ::1
This also fixes the problem Hajimu-san is seeing with pathmtu discovery
which I thought to reside in ip6_getpmtu().
Thank you for catching the bug!
--
Andre
More information about the freebsd-current
mailing list