aboutsummaryrefslogtreecommitdiff
path: root/print-mpcp.c
diff options
context:
space:
mode:
authorFrancois-Xavier Le Bail <devel.fx.lebail@orange.fr>2020-09-05 19:40:32 +0200
committerFrancois-Xavier Le Bail <devel.fx.lebail@orange.fr>2020-09-06 20:22:29 +0200
commitabc8c2d61c65757011d9d32abb01cd1e51232f3a (patch)
treea4c9890086314bffc5236c815c773975e471cc4a /print-mpcp.c
parent54b5db3dde658894be730b15480f9b6e27b717a0 (diff)
downloadtcpdump-abc8c2d61c65757011d9d32abb01cd1e51232f3a.tar.gz
Remove many (762) now redundant ND_TCHECK_n() calls
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-mpcp.c')
-rw-r--r--print-mpcp.c2
1 files changed, 0 insertions, 2 deletions
diff --git a/print-mpcp.c b/print-mpcp.c
index 47b2a107..b5171f9a 100644
--- a/print-mpcp.c
+++ b/print-mpcp.c
@@ -176,7 +176,6 @@ mpcp_print(netdissect_options *ndo, const u_char *pptr, u_int length)
tptr += sizeof(struct mpcp_grant_t);
}
- ND_TCHECK_2(tptr);
ND_PRINT("\n\tSync-Time %u ticks", GET_BE_U_2(tptr));
break;
@@ -198,7 +197,6 @@ mpcp_print(netdissect_options *ndo, const u_char *pptr, u_int length)
report=1;
while (report_bitmap != 0) {
if (report_bitmap & 1) {
- ND_TCHECK_2(tptr);
ND_PRINT("\n\t Q%u Report, Duration %u ticks",
report,
GET_BE_U_2(tptr));