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

Use KASSERTs as suggested by glebius@

Browse Source 
MFC after: 3 days
X-MFC with: 265691
This commit is contained in:
tuexen 2014-05-08 20:47:54 +00:00
parent 7e07adf133
commit b6b80cf495
1 changed files with 2 additions and 14 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
sys/netinet/ip_output.c
View File

@ -892,20 +892,8 @@ in_delayed_cksum(struct mbuf *m)
offset -= m->m_len;
m = m->m_next;
}
if (m == NULL) {
/* This should not happen. */
printf("in_delayed_cksum(): checksum outside mbuf chain.\n");
return;
}
if (offset + sizeof(u_short) > m->m_len) {
/*
* XXX
* This should not happen, but if it does, it might make more
* sense to fix the caller than to add code to split it here.
*/
printf("in_delayed_cksum(): checksum split between mbufs.\n");
return;
}
KASSERT(m != NULL, ("in_delayed_cksum: checksum outside mbuf chain."));
KASSERT(offset + sizeof(u_short) <= m->m_len, ("in_delayed_cksum: checksum split between mbufs."));
*(u_short *)(m->m_data + offset) = csum;
}