diff options
| author | rwatson <rwatson@FreeBSD.org> | 2009-04-19 22:29:16 +0000 |
|---|---|---|
| committer | rwatson <rwatson@FreeBSD.org> | 2009-04-19 22:29:16 +0000 |
| commit | ee528abe4691b42f196e602746c8b74a3fa38b00 (patch) | |
| tree | bdcf6894120fe9bc0b1be62743eadf5c634f7f83 /sys/netinet | |
| parent | 12c8c21e5c5cb2bd6ea6558dbca36e0993857f70 (diff) | |
| download | FreeBSD-src-ee528abe4691b42f196e602746c8b74a3fa38b00.zip FreeBSD-src-ee528abe4691b42f196e602746c8b74a3fa38b00.tar.gz | |
In divert_packet(), lock the interface address list before iterating over
it in search of an address.
MFC after: 2 weeks
Diffstat (limited to 'sys/netinet')
| -rw-r--r-- | sys/netinet/ip_divert.c | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/sys/netinet/ip_divert.c b/sys/netinet/ip_divert.c index f39c655..8a2fc25 100644 --- a/sys/netinet/ip_divert.c +++ b/sys/netinet/ip_divert.c @@ -244,18 +244,22 @@ divert_packet(struct mbuf *m, int incoming) divsrc.sin_port = divert_cookie(mtag); /* record matching rule */ if (incoming) { struct ifaddr *ifa; + struct ifnet *ifp; /* Sanity check */ M_ASSERTPKTHDR(m); /* Find IP address for receive interface */ - TAILQ_FOREACH(ifa, &m->m_pkthdr.rcvif->if_addrhead, ifa_link) { + ifp = m->m_pkthdr.rcvif; + IF_ADDR_LOCK(ifp); + TAILQ_FOREACH(ifa, &ifp->if_addrhead, ifa_link) { if (ifa->ifa_addr->sa_family != AF_INET) continue; divsrc.sin_addr = ((struct sockaddr_in *) ifa->ifa_addr)->sin_addr; break; } + IF_ADDR_UNLOCK(ifp); } /* * Record the incoming interface name whenever we have one. |
