summaryrefslogtreecommitdiffstats
path: root/sys/kern/uipc_mbuf.c
diff options
context:
space:
mode:
authorjulian <julian@FreeBSD.org>2002-09-19 08:28:41 +0000
committerjulian <julian@FreeBSD.org>2002-09-19 08:28:41 +0000
commitc431e46788ee8d257c6ea3b91a72c20a4f95c3a2 (patch)
treecfd09f3adf9a90d881d370979c2880126d5fc407 /sys/kern/uipc_mbuf.c
parent3935e48680c808fe65f3a9d6149df4ef37cdb40e (diff)
downloadFreeBSD-src-c431e46788ee8d257c6ea3b91a72c20a4f95c3a2.zip
FreeBSD-src-c431e46788ee8d257c6ea3b91a72c20a4f95c3a2.tar.gz
While well intentionned the check to see it there is a packet
header and return that length, was misguided. The check itself didn't take into account the fact that the mbuf pointer pased in may be null, and the function is defined specifically for cases where the caller knows what it wants. Rather than fix the check I'm removing it as phk suggested. Submitted by: phk@freebsd.org
Diffstat (limited to 'sys/kern/uipc_mbuf.c')
-rw-r--r--sys/kern/uipc_mbuf.c3
1 files changed, 0 insertions, 3 deletions
diff --git a/sys/kern/uipc_mbuf.c b/sys/kern/uipc_mbuf.c
index 847b4ee..cad6d3e 100644
--- a/sys/kern/uipc_mbuf.c
+++ b/sys/kern/uipc_mbuf.c
@@ -727,9 +727,6 @@ m_length(struct mbuf *m0, struct mbuf **last)
struct mbuf *m;
u_int len;
- if ((m0->m_flags & M_PKTHDR) != 0)
- return (m0->m_pkthdr.len);
-
len = 0;
for (m = m0; m != NULL; m = m->m_next) {
len += m->m_len;
OpenPOWER on IntegriCloud