diff options
| author | Francois-Xavier Le Bail <devel.fx.lebail@orange.fr> | 2020-08-10 16:22:37 +0200 |
|---|---|---|
| committer | Francois-Xavier Le Bail <devel.fx.lebail@orange.fr> | 2020-09-05 14:59:21 +0200 |
| commit | 929e4be621ebdcede37865a5767a175f083f9138 (patch) | |
| tree | bf6ecbc038696e66176b2d50e04e015767f23b3d /print-tipc.c | |
| parent | 54b5db3dde658894be730b15480f9b6e27b717a0 (diff) | |
| download | tcpdump-test-rc.tar.gz | |
Remove many (761) redundant ND_TCHECK_n() callstest-rc
ND_TCHECK_n(e), n in { 1, 2, 3, 4, 8 }.
They are redundant because they are followed by a GET_.*_n(e) call,
same n, same e, which do the bounds check.
Remove unused 'trunc' labels and most associated codes.
Update the outputs of some tests accordingly.
Diffstat (limited to 'print-tipc.c')
| -rw-r--r-- | print-tipc.c | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/print-tipc.c b/print-tipc.c index cd52d239..3a9897ef 100644 --- a/print-tipc.c +++ b/print-tipc.c @@ -189,7 +189,6 @@ print_payload(netdissect_options *ndo, const struct payload_tipc_pkthdr *ap) tok2str(tipcuser_values, "unknown", user), tok2str(tipcmtype_values, "Unknown", mtype)); } else { - ND_TCHECK_4(ap->dest_node); orig_node = GET_BE_U_4(ap->orig_node); dest_node = GET_BE_U_4(ap->dest_node); ND_PRINT("TIPC v%u.0 %u.%u.%u:%u > %u.%u.%u:%u, headerlength %u bytes, MessageSize %u bytes, %s, messageType %s", @@ -342,7 +341,6 @@ tipc_print(netdissect_options *ndo, const u_char *bp, u_int length _U_, ndo->ndo_protocol = "tipc"; ap = (const struct tipc_pkthdr *)bp; - ND_TCHECK_4(ap->w0); w0 = GET_BE_U_4(ap->w0); user = TIPC_USER(w0); @@ -370,8 +368,4 @@ tipc_print(netdissect_options *ndo, const u_char *bp, u_int length _U_, break; } - return; - -trunc: - nd_print_trunc(ndo); } |