From c5f4e4d146f6f57a66bd4d7792e8ccf9625d039c Mon Sep 17 00:00:00 2001 From: David Gibson Date: Wed, 6 Nov 2024 12:43:04 +1100 Subject: [PATCH] fwd: Squash different-signedness comparison warning On certain architectures we get a warning about comparison between different signedness integers in fwd_probe_ephemeral(). This is because NUM_PORTS evaluates to an unsigned integer. It's a fixed value, though and we know it will fit in a signed long on anything reasonable, so add a cast to suppress the warning. Signed-off-by: David Gibson Signed-off-by: Stefano Brivio --- fwd.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/fwd.c b/fwd.c index c71f5e1..0b7f8b1 100644 --- a/fwd.c +++ b/fwd.c @@ -75,8 +75,8 @@ void fwd_probe_ephemeral(void) if (*end || errno) goto parse_err; - if (min < 0 || min >= NUM_PORTS || - max < 0 || max >= NUM_PORTS) + if (min < 0 || min >= (long)NUM_PORTS || + max < 0 || max >= (long)NUM_PORTS) goto parse_err; fwd_ephemeral_min = min;