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

On some interfaces, ipfilter drops UDP packets with zero checkum.

Browse Source 
This commit fixes that.

PR:		166372
Submitted by:	mk@neon1.net
Reviewed by:	Darren Reed <darrenr@reed.wattle.id.au>
MFC after:	1 week
This commit is contained in:
Cy Schubert 2015-10-06 03:41:11 +00:00
parent db0d64f24a
commit d029a42ab4
1 changed files with 16 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
sys/contrib/ipfilter/netinet/ip_fil_freebsd.c
View File

@ -1104,6 +1104,22 @@ ipf_checkv4sum(fin)
return -1;
}
if (m->m_pkthdr.csum_flags & CSUM_DATA_VALID) {
/* Depending on the driver, UDP may have zero checksum */
if (fin->fin_p == IPPROTO_UDP && (fin->fin_flx &
(FI_FRAG|FI_SHORT|FI_BAD)) == 0) {
udphdr_t *udp = fin->fin_dp;
if (udp->uh_sum == 0) {
/*
* we're good no matter what the hardware
* checksum flags and csum_data say (handling
* of csum_data for zero UDP checksum is not
* consistent across all drivers)
*/
fin->fin_cksum = 1;
return 0;
}
}
if (m->m_pkthdr.csum_flags & CSUM_PSEUDO_HDR)
sum = m->m_pkthdr.csum_data;
else