diff options
| author | Denis Ovsienko <denis@ovsienko.info> | 2016-11-01 17:05:01 +0000 |
|---|---|---|
| committer | Francois-Xavier Le Bail <fx.lebail@yahoo.com> | 2017-01-18 09:16:41 +0100 |
| commit | 4804e66125cb4a2bfc186796b30e7120925af4d6 (patch) | |
| tree | 4e4e9d663873d8e1b1cee601bec842b93e57a7f6 /tests/tcp-auth-heapoverflow.out | |
| parent | e9ac8b2c8588852576ef74f71338eceb1d820596 (diff) | |
| download | tcpdump-4804e66125cb4a2bfc186796b30e7120925af4d6.tar.gz | |
TCP: put TCP-AO option decoding right
As it was correctly pointed out in GitHub issue #516, the TCPOPT_TCPAO
(formerly TCPOPT_AUTH) case had an issue with option length processing,
though without significant consequences thanks to a check elsewhere.
Besides that, the old code (introduced in 2005) decoded a structure
similar to a proposed encoding variant of the early (first published in
2007) revisions of the Internet-Draft but different from the encoding
of RFC 5925 (published in 2010). These issues are now addressed and the
TCP option renamed to TCP-AO.
Diffstat (limited to 'tests/tcp-auth-heapoverflow.out')
| -rw-r--r-- | tests/tcp-auth-heapoverflow.out | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tests/tcp-auth-heapoverflow.out b/tests/tcp-auth-heapoverflow.out index bbf12a08..b7ff7f7a 100644 --- a/tests/tcp-auth-heapoverflow.out +++ b/tests/tcp-auth-heapoverflow.out @@ -1,2 +1,2 @@ IP (tos 0x30, ttl 48, id 12336, offset 0, flags [DF], proto TCP (6), length 12336, bad cksum 3030 (->29a8)!) - 48.48.48.48.12336 > 48.48.48.48.12336: Flags [.U], seq 808464432:808476696, ack 808464432, win 12336, urg 12336, options [enhanced auth keyid 4830303030303030[|tcp] + 48.48.48.48.12336 > 48.48.48.48.12336: Flags [.U], seq 808464432:808476696, ack 808464432, win 12336, urg 12336, options [tcp-ao keyid 48 rnextkeyid 48 mac 0x303030303030[|tcp] |