]> The Tcpdump Group git mirrors - tcpdump/commitdiff
RX: Remove 1 now redundant ND_TCHECK_LEN(e, sizeof(uint64_t)) call
authorFrancois-Xavier Le Bail <[email protected]>
Wed, 30 Sep 2020 11:12:05 +0000 (13:12 +0200)
committerFrancois-Xavier Le Bail <[email protected]>
Wed, 30 Sep 2020 11:21:17 +0000 (13:21 +0200)
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.

print-rx.c

index 3b7a623dde543e0e07a7a334fafedc9d99b9c6ba..4f0195282ff6340db97a2f2bd23ef74225e02d28 100644 (file)
@@ -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); \