diff options
| author | Francois-Xavier Le Bail <devel.fx.lebail@orange.fr> | 2020-09-07 11:22:49 +0200 |
|---|---|---|
| committer | Francois-Xavier Le Bail <devel.fx.lebail@orange.fr> | 2020-09-08 14:35:21 +0200 |
| commit | 3b8ff4bf0d8f389e00c19d6c094bce2bbf00deca (patch) | |
| tree | c92ecb6d42a8bd774c3f864124d693d0657c90d7 /print-isoclns.c | |
| parent | aad6ac30ce7904f688226dbc61021ca4f907274f (diff) | |
| download | tcpdump-3b8ff4bf0d8f389e00c19d6c094bce2bbf00deca.tar.gz | |
Remove some now redundant ND_TCHECK_LEN(e, sizeof(nd_ipv4)) calls
They are redundant because they are followed by a GET_IPADDR_STRING(e)
call, same e, which do the bounds check.
Remove unused 'trunc' label(s) and associated code(s).
Diffstat (limited to 'print-isoclns.c')
| -rw-r--r-- | print-isoclns.c | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/print-isoclns.c b/print-isoclns.c index 288c4b18..e842a008 100644 --- a/print-isoclns.c +++ b/print-isoclns.c @@ -3236,7 +3236,6 @@ isis_print(netdissect_options *ndo, case ISIS_TLV_TE_ROUTER_ID: if (tlen < sizeof(nd_ipv4)) goto tlv_trunc; - ND_TCHECK_LEN(pptr, sizeof(nd_ipv4)); ND_PRINT("\n\t Traffic Engineering Router ID: %s", GET_IPADDR_STRING(pptr)); break; @@ -3244,7 +3243,6 @@ isis_print(netdissect_options *ndo, while (tlen != 0) { if (tlen < sizeof(nd_ipv4)) goto tlv_trunc; - ND_TCHECK_LEN(tptr, sizeof(nd_ipv4)); ND_PRINT("\n\t IPv4 interface address: %s", GET_IPADDR_STRING(tptr)); tptr += sizeof(nd_ipv4); tlen -= sizeof(nd_ipv4); @@ -3274,14 +3272,12 @@ isis_print(netdissect_options *ndo, if (tlen < sizeof(nd_ipv4)) break; - ND_TCHECK_LEN(tptr, sizeof(nd_ipv4)); ND_PRINT("\n\t IPv4 interface address: %s", GET_IPADDR_STRING(tptr)); tptr+=sizeof(nd_ipv4); tlen-=sizeof(nd_ipv4); if (tlen < sizeof(nd_ipv4)) break; - ND_TCHECK_LEN(tptr, sizeof(nd_ipv4)); ND_PRINT("\n\t IPv4 neighbor address: %s", GET_IPADDR_STRING(tptr)); tptr+=sizeof(nd_ipv4); tlen-=sizeof(nd_ipv4); |