diff options
author | Christian Brabandt <cb@256bit.org> | 2024-01-12 17:48:08 +0100 |
---|---|---|
committer | Christian Brabandt <cb@256bit.org> | 2024-01-12 17:48:08 +0100 |
commit | 49471963fefbdf78239d9066d84e14e1876fb177 (patch) | |
tree | cfedd90073809d663495af4f8ceaeaa351c5c13b | |
parent | 424ec1f235a53131042aed94d30f6e2528e551d9 (diff) |
patch 9.1.0022: Coverity complains about improper use of negative valuev9.1.0022
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 <cb@256bit.org>
-rw-r--r-- | src/cindent.c | 3 | ||||
-rw-r--r-- | src/version.c | 2 |
2 files changed, 4 insertions, 1 deletions
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 @@ -705,6 +705,8 @@ static char *(features[]) = static int included_patches[] = { /* Add new patch number below this line */ /**/ + 22, +/**/ 21, /**/ 20, |