From da0326ffc6c13a2470905b74f8fbe75049ce28e4 Mon Sep 17 00:00:00 2001 From: weichslgartner Date: Thu, 30 Jun 2022 18:32:19 +0200 Subject: [PATCH] fix for issue #369 adding cast preventing int/unsigned comparison (-Wsign-compare warning in lib.c) --- lib.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/lib.c b/lib.c index 81fe64b..3c1a0d9 100644 --- a/lib.c +++ b/lib.c @@ -572,13 +572,13 @@ static int snprintf_error_data(char *buf, size_t len, uint8_t err, if (count){ /* Fix for potential buffer overflow https://lgtm.com/rules/1505913226124/ */ tmp_n = snprintf(buf + n, len - n, ","); - if (tmp_n < 0 || tmp_n >= len - n){ + if (tmp_n < 0 || (size_t)tmp_n >= len - n){ return n; } n += tmp_n; } tmp_n = snprintf(buf + n, len - n, "%s", arr[i]); - if (tmp_n < 0 || tmp_n >= len - n){ + if (tmp_n < 0 || (size_t)tmp_n >= len - n){ return n; } n += tmp_n; @@ -658,13 +658,13 @@ void snprintf_can_error_frame(char *buf, size_t len, const struct canfd_frame *c if (classes){ /* Fix for potential buffer overflow https://lgtm.com/rules/1505913226124/ */ tmp_n = snprintf(buf + n, len - n, "%s", sep); - if (tmp_n < 0 || tmp_n >= len - n){ + if (tmp_n < 0 || (size_t)tmp_n >= len - n){ return; } n += tmp_n; } tmp_n = snprintf(buf + n, len - n, "%s", error_classes[i]); - if (tmp_n < 0 || tmp_n >= len - n){ + if (tmp_n < 0 || (size_t)tmp_n >= len - n){ return; } n += tmp_n;