Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 31 Oct 2007 04:39:38 GMT
From:      Kip Macy <kmacy@FreeBSD.org>
To:        Perforce Change Reviews <perforce@freebsd.org>
Subject:   PERFORCE change 128386 for review
Message-ID:  <200710310439.l9V4dckw098725@repoman.freebsd.org>

next in thread | raw e-mail | index | archive | help
http://perforce.freebsd.org/chv.cgi?CH=128386

Change 128386 by kmacy@kmacy:storage:toestack on 2007/10/31 04:39:12

	use RTRFREE_LOCKED in case their an external reference to the route

Affected files ...

.. //depot/projects/toestack/sys/netinet/if_ether.c#9 edit

Differences ...

==== //depot/projects/toestack/sys/netinet/if_ether.c#9 (text+ko) ====

@@ -810,12 +810,12 @@
 			 * over who claims what Ether address.
 			 */
 			if (rt->rt_ifp == ifp) {
-				rtfree(rt);
+				RTFREE_LOCKED(rt);
 				goto drop;
 			}
 			(void)memcpy(ar_tha(ah), ar_sha(ah), ah->ar_hln);
 			(void)memcpy(ar_sha(ah), enaddr, ah->ar_hln);
-			rtfree(rt);
+			RTFREE_LOCKED(rt);
 
 			/*
 			 * Also check that the node which sent the ARP packet
@@ -833,10 +833,10 @@
 				    " from %s via %s, expecting %s\n",
 				    inet_ntoa(isaddr), ifp->if_xname,
 				    rt->rt_ifp->if_xname);
-				rtfree(rt);
+				RTFREE_LOCKED(rt);
 				goto drop;
 			}
-			rtfree(rt);
+			RTFREE_LOCKED(rt);
 
 #ifdef DEBUG_PROXY
 			printf("arp: proxying for %s\n",



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200710310439.l9V4dckw098725>