From: Francois-Xavier Le Bail <devel.fx.lebail@orange.fr>
Date: Wed, 30 Sep 2020 11:12:05 +0000 (+0200)
Subject: RX: Remove 1 now redundant ND_TCHECK_LEN(e, sizeof(uint64_t)) call
X-Git-Tag: tcpdump-4.99-bp~180
X-Git-Url: https://git.tcpdump.org/tcpdump/commitdiff_plain/1fa77a87dc53451189525b79fff72a0de13be322?ds=inline

RX: Remove 1 now redundant ND_TCHECK_LEN(e, sizeof(uint64_t)) call

It is redundant because it is identical to ND_TCHECK_8(e), followed
by a GET_BE_U_8(e), same e, which do the bounds check.
---

diff --git a/print-rx.c b/print-rx.c
index 3b7a623d..4f019528 100644
--- a/print-rx.c
+++ b/print-rx.c
@@ -790,7 +790,6 @@ rx_cache_find(netdissect_options *ndo, const struct rx_header *rxh,
 		}
 
 #define UINT64OUT() { uint64_t _i; \
-			ND_TCHECK_LEN(bp, sizeof(uint64_t)); \
 			_i = GET_BE_U_8(bp); \
 			bp += sizeof(uint64_t); \
 			ND_PRINT(" %" PRIu64, _i); \