Update the ND_TCHECK_LEN macro definitions

Avoid warnings such as in the following case:

        if (...)
                ND_TCHECK_LEN(...);
        else

With gcc:
source.c:X:12: warning: suggest explicit braces to avoid ambiguous
  'else' [-Wdangling-else]
    X |         if (...)
      |            ^

With clang:
source.c:Y:2: warning: add explicit braces to avoid dangling else
  [-Wdangling-else]
        else
        ^

(cherry picked from commit 6fa624334b)
This commit is contained in:
Francois-Xavier Le Bail 2024-10-15 21:08:57 +02:00
parent e28a04703f
commit d477c4264a

View File

@ -374,9 +374,15 @@ NORETURN void nd_trunc_longjmp(netdissect_options *ndo);
/* Bail out if "l" bytes from "p" were not captured */
#ifdef ND_LONGJMP_FROM_TCHECK
#define ND_TCHECK_LEN(p, l) if (!ND_TTEST_LEN(p, l)) nd_trunc_longjmp(ndo)
#define ND_TCHECK_LEN(p, l) \
do { \
if (!ND_TTEST_LEN(p, l)) nd_trunc_longjmp(ndo); \
} while (0)
#else
#define ND_TCHECK_LEN(p, l) if (!ND_TTEST_LEN(p, l)) goto trunc
#define ND_TCHECK_LEN(p, l) \
do { \
if (!ND_TTEST_LEN(p, l)) goto trunc; \
} while (0)
#endif
/* Bail out if "*(p)" was not captured */