diff options
author | dg <dg@FreeBSD.org> | 1995-04-01 01:46:27 +0000 |
---|---|---|
committer | dg <dg@FreeBSD.org> | 1995-04-01 01:46:27 +0000 |
commit | dfeb70b07153029e9e0f71f393f3ccc70deecffe (patch) | |
tree | a72b69223277cb4d95eae7ef9a7241ce31b5ea35 /sys/net | |
parent | e7f6953c5ba8b72d9a2264f4f4b916c26056029b (diff) | |
download | FreeBSD-src-dfeb70b07153029e9e0f71f393f3ccc70deecffe.zip FreeBSD-src-dfeb70b07153029e9e0f71f393f3ccc70deecffe.tar.gz |
Patch from Greg Ansley:
In rare cases, when the filter specified accesses an multi-byte value that
is split across mbuf's, the value loaded is incorrect. And if you are very
unlucky (like me) it will index off the end of the mbuf and into an
unallocated page and panic the system.
If you look at the code you will discover the the index *k* is added to
the pointer *cp* and the used AGAIN as a subscript.
Diffstat (limited to 'sys/net')
-rw-r--r-- | sys/net/bpf_filter.c | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/sys/net/bpf_filter.c b/sys/net/bpf_filter.c index 8265410..b655dda2 100644 --- a/sys/net/bpf_filter.c +++ b/sys/net/bpf_filter.c @@ -37,7 +37,7 @@ * * @(#)bpf_filter.c 8.1 (Berkeley) 6/10/93 * - * $Id$ + * $Id: bpf_filter.c,v 1.3 1994/08/02 07:45:58 davidg Exp $ */ #include <sys/param.h> @@ -112,14 +112,14 @@ m_xword(m, k, err) switch (len - k) { case 1: - return (cp[k] << 24) | (np[0] << 16) | (np[1] << 8) | np[2]; + return (cp[0] << 24) | (np[0] << 16) | (np[1] << 8) | np[2]; case 2: - return (cp[k] << 24) | (cp[k + 1] << 16) | (np[0] << 8) | + return (cp[0] << 24) | (cp[1] << 16) | (np[0] << 8) | np[1]; default: - return (cp[k] << 24) | (cp[k + 1] << 16) | (cp[k + 2] << 8) | + return (cp[0] << 24) | (cp[1] << 16) | (cp[2] << 8) | np[0]; } bad: @@ -153,7 +153,7 @@ m_xhalf(m, k, err) if (m0 == 0) goto bad; *err = 0; - return (cp[k] << 8) | mtod(m0, u_char *)[0]; + return (cp[0] << 8) | mtod(m0, u_char *)[0]; bad: *err = 1; return 0; |