From: Christian Brabandt Date: Fri, 12 Jan 2024 16:48:08 +0000 (+0100) Subject: patch 9.1.0022: Coverity complains about improper use of negative value X-Git-Tag: v9.1.0022^0 X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=49471963fefbdf78239d9066d84e14e1876fb177;p=thirdparty%2Fvim.git patch 9.1.0022: Coverity complains about improper use of negative value Problem: Coverity complains about improper use of negative value Solution: Add a condition to validate that keytyped is larger or equal to 0 Apparently patch 9.1.0006 made it more explicit for Coverity, that the TOLOWER_LOC() macros do not handle negative values properly. However, that condition has always been there even before that, so add a condition to verify that keytyped is indeed at least 0 closes: #13824 Signed-off-by: Christian Brabandt --- diff --git a/src/cindent.c b/src/cindent.c index 62894228b1..cf18441f60 100644 --- a/src/cindent.c +++ b/src/cindent.c @@ -4108,7 +4108,8 @@ in_cinkeys( } else // TODO: multi-byte - if (keytyped == (int)p[-1] || (icase && keytyped < 256 + if (keytyped == (int)p[-1] || (icase + && keytyped < 256 && keytyped >= 0 && TOLOWER_LOC(keytyped) == TOLOWER_LOC((int)p[-1]))) { line = ml_get_cursor(); diff --git a/src/version.c b/src/version.c index b221faaae0..562ba1317d 100644 --- a/src/version.c +++ b/src/version.c @@ -704,6 +704,8 @@ static char *(features[]) = static int included_patches[] = { /* Add new patch number below this line */ +/**/ + 22, /**/ 21, /**/