diff options
author | Francois-Xavier Le Bail <devel.fx.lebail@orange.fr> | 2020-09-05 21:56:50 +0200 |
---|---|---|
committer | Francois-Xavier Le Bail <devel.fx.lebail@orange.fr> | 2020-09-08 14:14:13 +0200 |
commit | aad6ac30ce7904f688226dbc61021ca4f907274f (patch) | |
tree | 76ffb3367a34d7e5225fa1bc0a75aa6406118b6d /print-ppp.c | |
parent | cbe18055349fa44ac8348c56834c32586e0a81ab (diff) | |
download | tcpdump-aad6ac30ce7904f688226dbc61021ca4f907274f.tar.gz |
Remove some now redundant ND_TCHECK_4() calls
ND_TCHECK_4(e).
They are redundant because they are followed by a GET_IPADDR_STRING(e)
call, same e, which do the bounds check.
Remove unused 'trunc' labels and associated codes.
Update the output of a test accordingly.
Diffstat (limited to 'print-ppp.c')
-rw-r--r-- | print-ppp.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/print-ppp.c b/print-ppp.c index 759b06b1..fe036247 100644 --- a/print-ppp.c +++ b/print-ppp.c @@ -731,7 +731,6 @@ print_lcp_config_options(netdissect_options *ndo, ND_PRINT(" (length bogus, should be = 7)"); return 0; } - ND_TCHECK_4(p + 3); ND_PRINT(": IPv4 %s", GET_IPADDR_STRING(p + 3)); break; case MEDCLASS_MAC: @@ -1038,7 +1037,6 @@ print_ipcp_config_options(netdissect_options *ndo, ND_PRINT(" (length bogus, should be = 10)"); return len; } - ND_TCHECK_4(p + 6); ND_PRINT(": src %s, dst %s", GET_IPADDR_STRING(p + 2), GET_IPADDR_STRING(p + 6)); @@ -1124,7 +1122,6 @@ print_ipcp_config_options(netdissect_options *ndo, ND_PRINT(" (length bogus, should be = 6)"); return 0; } - ND_TCHECK_4(p + 2); ND_PRINT(": %s", GET_IPADDR_STRING(p + 2)); break; default: |