d/freebsd-nq
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

In in_pcbconnect_setup(), don't use the cached inp->inp_route unless

Browse Source 
it is marked as RTF_UP. This appears to fix a crash that was sometimes
triggered when dhclient(8) tried to send a packet after an interface
had been detatched.

Reviewed by:	sam
This commit is contained in:
Ian Dowse 2003-11-10 22:45:37 +00:00
parent dab12c8eba
commit 3ab2096b80
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
sys/netinet/in_pcb.c
View File

@ -578,10 +578,10 @@ in_pcbconnect_setup(inp, nam, laddrp, lportp, faddrp, fportp, oinpp, td)
* destination, in case of sharing the cache with IPv6. * destination, in case of sharing the cache with IPv6.
*/ */
ro = &inp->inp_route; ro = &inp->inp_route;
if (ro->ro_rt && if (ro->ro_rt && ((ro->ro_rt->rt_flags & RTF_UP) == 0 ||
(ro->ro_dst.sa_family != AF_INET || ro->ro_dst.sa_family != AF_INET ||
satosin(&ro->ro_dst)->sin_addr.s_addr != faddr.s_addr || satosin(&ro->ro_dst)->sin_addr.s_addr != faddr.s_addr ||
inp->inp_socket->so_options & SO_DONTROUTE)) { inp->inp_socket->so_options & SO_DONTROUTE)) {
RTFREE(ro->ro_rt); RTFREE(ro->ro_rt);
ro->ro_rt = (struct rtentry *)0; ro->ro_rt = (struct rtentry *)0;
} }