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-forces.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-forces.c')
| -rw-r--r-- | print-forces.c | 11 |
1 files changed, 0 insertions, 11 deletions
diff --git a/print-forces.c b/print-forces.c index 12f6016d..8d398509 100644 --- a/print-forces.c +++ b/print-forces.c @@ -1239,7 +1239,6 @@ asttlv_print(netdissect_options *ndo, ND_PRINT("illegal ASTresult-TLV: %u bytes!\n", dlen); return -1; } - ND_TCHECK_4(pptr); rescode = GET_BE_U_4(pptr); if (rescode > ASTMCD) { ND_PRINT("illegal ASTresult result code: %u!\n", rescode); @@ -1271,10 +1270,6 @@ asttlv_print(netdissect_options *ndo, ND_PRINT("(%x)\n%s", rescode, ib); } return 0; - -trunc: - nd_print_trunc(ndo); - return -1; } #define ASRDLN 4 @@ -1297,7 +1292,6 @@ asrtlv_print(netdissect_options *ndo, ND_PRINT("illegal ASRresult-TLV: %u bytes!\n", dlen); return -1; } - ND_TCHECK_4(pptr); rescode = GET_BE_U_4(pptr); if (rescode > ASRMCD) { @@ -1324,10 +1318,6 @@ asrtlv_print(netdissect_options *ndo, ND_PRINT("(%x)\n%s", rescode, ib); } return 0; - -trunc: - nd_print_trunc(ndo); - return -1; } #if 0 @@ -1730,7 +1720,6 @@ forces_print(netdissect_options *ndo, goto error; } - ND_TCHECK_4(pptr + 20); flg_raw = GET_BE_U_4(pptr + 20); if (ndo->ndo_vflag >= 1) { ND_PRINT("\n\tForCES Version %u len %uB flags 0x%08x ", |