diff options
author | Denis Ovsienko <denis@ovsienko.info> | 2020-09-16 01:38:07 +0100 |
---|---|---|
committer | Denis Ovsienko <denis@ovsienko.info> | 2020-09-16 01:38:07 +0100 |
commit | 503c7d1dc16961fdd8a3f62aea688cab270328e9 (patch) | |
tree | 27dda3be500f5d881fdfd8ec46cf3c380b5ee59c /print-ipfc.c | |
parent | 0a5faecf953a8a997b0e90e0bed1d763542166e0 (diff) | |
download | tcpdump-503c7d1dc16961fdd8a3f62aea688cab270328e9.tar.gz |
OpenFlow 1.0: Lose 7 ND_TCHECK_LEN() calls.
Considering a protocol field that has a known size and contains a
zero-padded string, in a code of this particular form:
ND_TCHECK_LEN(field_start, field_length);
nd_print(ndo, field_start, field_start + field_length);
...the only effect of the ND_TCHECK_LEN() call is that when the protocol
field is not entirely within the packet buffer, the code before
triggering a longjmp() prints nothing instead of printing the part of
the field that is within the packet buffer. Thus remove these instances
of ND_TCHECK_LEN(), so the output better represents the captured data.
Diffstat (limited to 'print-ipfc.c')
0 files changed, 0 insertions, 0 deletions