diff options
| author | Francois-Xavier Le Bail <devel.fx.lebail@orange.fr> | 2018-01-25 20:36:16 +0100 |
|---|---|---|
| committer | Francois-Xavier Le Bail <devel.fx.lebail@orange.fr> | 2018-02-07 18:48:49 +0100 |
| commit | f253e110096ed673dc9de26da43ca80eaa2baec1 (patch) | |
| tree | 653cd4d65e298345866b9020cc1a737384fc97a3 /tests/l2tp-avp-overflow.out | |
| parent | fb154b00bad613756c18c56dd01bbb37e78f1b22 (diff) | |
| download | tcpdump-f253e110096ed673dc9de26da43ca80eaa2baec1.tar.gz | |
Add a sanity check on packet header length
The packet length must be <= MAXIMUM_SNAPLEN.
Currently, there is no D-Bus printer, thus no need for a bigger length.
Now a pachet is valid if:
capture length != 0,
packet length != 0,
capture length <= MAXIMUM_SNAPLEN,
packet length <= MAXIMUM_SNAPLEN,
packet length >= capture length.
Moreover:
Fix the packet header lengths of some fuzzed pcap files:
If the lengths are > MAXIMUM_SNAPLEN, set them to MAXIMUM_SNAPLEN.
Thus they will be always usable with this new sanity check.
Diffstat (limited to 'tests/l2tp-avp-overflow.out')
| -rw-r--r-- | tests/l2tp-avp-overflow.out | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/tests/l2tp-avp-overflow.out b/tests/l2tp-avp-overflow.out index a3b1db2d..9ed38674 100644 --- a/tests/l2tp-avp-overflow.out +++ b/tests/l2tp-avp-overflow.out @@ -36,4 +36,3 @@ IP (tos 0x30, ttl 48, id 12331, offset 0, flags [none], proto UDP (17), length 8 127.0.0.229.12416 > 127.236.0.1.1701: l2tp:[TL](560/2056) VENDOR0001:ATTR0023(0530) |... IP (tos 0x30, ttl 48, id 12336, offset 0, flags [none], proto UDP (17), length 12336, bad cksum 1f51 (->2a8b)!) 127.0.0.229.12416 > 127.236.0.1.1701: l2tp:[TL](560/2056) VENDOR0080:ATTR06a5(19e8) |... -EXIT CODE 00000100 |