]> git.ipfire.org Git - thirdparty/haproxy.git/commitdiff
BUILD: cli: appease a null-deref warning in cli_gen_usage_msg()
authorWilly Tarreau <w@1wt.eu>
Mon, 10 May 2021 05:47:05 +0000 (07:47 +0200)
committerWilly Tarreau <w@1wt.eu>
Mon, 10 May 2021 05:47:05 +0000 (07:47 +0200)
The compiler sees the possibility of null-deref for which a path is
possible but which doesn't exist as we didn't pass a null args outside
of the help request. The test was introduced by the simplified test on
ishelp variable, so let's add it to shut the warning.

src/cli.c

index c2464b9e25695d118103436dd8eb0ee58d40b2dc..24144ddeddc769293adaecbf3189a421323059b9 100644 (file)
--- a/src/cli.c
+++ b/src/cli.c
@@ -143,7 +143,7 @@ static char *cli_gen_usage_msg(struct appctx *appctx, char * const *args)
        chunk_reset(tmp);
        if (ishelp) // this is the help message.
                chunk_strcat(tmp, "The following commands are valid at this level:\n");
-       else if (!length && (!*args || !**args)) // no match
+       else if (!length && (!args || !*args || !**args)) // no match
                chunk_strcat(tmp, "Unknown command. Please enter one of the following commands only:\n");
        else // partial match
                chunk_strcat(tmp, "Unknown command, but maybe one of the following ones is a better match:\n");