Gitweb:
https://sourceware.org/git/?p=lvm2.git;a=commitdiff;h=178df1c8f4aa6fa73f9...
Commit: 178df1c8f4aa6fa73f9f55f2a7f5f7b8c8c0964d
Parent: 462ac47003e14975b91832f19bc287bd1a775bb4
Author: Zdenek Kabelac <zkabelac(a)redhat.com>
AuthorDate: Tue Sep 21 18:32:21 2021 +0200
Committer: Zdenek Kabelac <zkabelac(a)redhat.com>
CommitterDate: Tue Sep 21 21:03:47 2021 +0200
gcc-fanalyzer: helping understand answer is not null
gcc can't easily see pointer 'answer' is only being checked once 'ret >
0'.
---
lib/display/display.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/lib/display/display.c b/lib/display/display.c
index f9c9ef836..3c2c2c2bc 100644
--- a/lib/display/display.c
+++ b/lib/display/display.c
@@ -974,7 +974,7 @@ char yes_no_prompt(const char *prompt, ...)
c = tolower(c);
- if ((ret > 0) && (c == answer[0]))
+ if ((ret > 0) && answer && (c == answer[0]))
answer++; /* Matching, next char */
else if (c == '\n') {
if (feof(stdin))
@@ -1000,7 +1000,7 @@ char yes_no_prompt(const char *prompt, ...)
/* Ignore any whitespace before */
--i;
goto nextchar;
- } else if ((ret > 0) && isspace(c)) {
+ } else if ((ret > 0) && answer && isspace(c)) {
/* Ignore any whitespace after */
while (*answer)
answer++; /* jump to end-of-word */