diff options
author | David S. Miller <davem@davemloft.net> | 2010-12-21 21:43:16 +0100 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2010-12-21 21:43:16 +0100 |
commit | da521b2c4f046383bc8941604174bc0e8bffb430 (patch) | |
tree | e3678d5af507054c86f1d913ad1599f4afee1c7d /crypto | |
parent | net_sched: sch_sfq: fix allot handling (diff) | |
download | linux-da521b2c4f046383bc8941604174bc0e8bffb430.tar.xz linux-da521b2c4f046383bc8941604174bc0e8bffb430.zip |
net: Fix range checks in tcf_valid_offset().
This function has three bugs:
1) The offset should be valid most of the time, this is just
a sanity check, therefore we should use "likely" not "unlikely"
2) This is the only place where we can check for arithmetic overflow
of the pointer plus the length.
3) The existing range checks are off by one, the valid range is
skb->head to skb_tail_pointer(), inclusive.
Based almost entirely upon a patch by Ralph Loader.
Reported-by: Ralph Loader <suckfish@ihug.co.nz>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'crypto')
0 files changed, 0 insertions, 0 deletions