Relocate the KASSERT for a null recvif to a location where it will
actually do some good. Pointed out by: ru
This commit is contained in:
parent
06279b5357
commit
807b8338ba
@ -607,6 +607,7 @@ icmp_reflect(m)
|
|||||||
LIST_FOREACH(ia, INADDR_HASH(t.s_addr), ia_hash)
|
LIST_FOREACH(ia, INADDR_HASH(t.s_addr), ia_hash)
|
||||||
if (t.s_addr == IA_SIN(ia)->sin_addr.s_addr)
|
if (t.s_addr == IA_SIN(ia)->sin_addr.s_addr)
|
||||||
goto match;
|
goto match;
|
||||||
|
KASSERT(m->m_pkthdr.rcvif != NULL, ("icmp_reflect: NULL rcvif"));
|
||||||
if (m->m_pkthdr.rcvif->if_flags & IFF_BROADCAST) {
|
if (m->m_pkthdr.rcvif->if_flags & IFF_BROADCAST) {
|
||||||
TAILQ_FOREACH(ifa, &m->m_pkthdr.rcvif->if_addrhead, ifa_link) {
|
TAILQ_FOREACH(ifa, &m->m_pkthdr.rcvif->if_addrhead, ifa_link) {
|
||||||
if (ifa->ifa_addr->sa_family != AF_INET)
|
if (ifa->ifa_addr->sa_family != AF_INET)
|
||||||
@ -617,7 +618,6 @@ icmp_reflect(m)
|
|||||||
goto match;
|
goto match;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
KASSERT(m->m_pkthdr.rcvif != NULL, ("icmp_reflect: NULL rcvif"));
|
|
||||||
icmpdst.sin_addr = t;
|
icmpdst.sin_addr = t;
|
||||||
ia = (struct in_ifaddr *)ifaof_ifpforaddr(
|
ia = (struct in_ifaddr *)ifaof_ifpforaddr(
|
||||||
(struct sockaddr *)&icmpdst, m->m_pkthdr.rcvif);
|
(struct sockaddr *)&icmpdst, m->m_pkthdr.rcvif);
|
||||||
|
Loading…
x
Reference in New Issue
Block a user