From cc4a92b13c08f500c1dc0c7af95af599b1c15095 Mon Sep 17 00:00:00 2001 From: Zdenek Kabelac Date: Tue, 25 Jun 2019 15:29:26 +0200 Subject: [PATCH] cov: ensure cname exists before derefering it Just make it clear to analyzers cname can't be NULL. TODO: maybe exclude NULL at front of the function... --- tools/command.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/tools/command.c b/tools/command.c index 724040e39..2e69effcb 100644 --- a/tools/command.c +++ b/tools/command.c @@ -1935,7 +1935,7 @@ void print_usage(struct command *cmd, int longhelp, int desc_first) * see print_common_options_cmd() */ - if ((cname->variants > 1) && cname->common_options[opt_enum]) + if (cname && (cname->variants > 1) && cname->common_options[opt_enum]) continue; printf("\n\t["); @@ -1975,7 +1975,7 @@ void print_usage(struct command *cmd, int longhelp, int desc_first) * see print_common_options_cmd() */ - if ((cname->variants > 1) && cname->common_options[opt_enum]) + if (cname && (cname->variants > 1) && cname->common_options[opt_enum]) continue; printf("\n\t["); @@ -3438,7 +3438,7 @@ static int _print_man(char *name, char *des_file, int secondary) if (!prev_cmd || strcmp(prev_cmd->name, cmd->name)) { printf(".SH NAME\n"); - if (cname->desc) + if (cname && cname->desc) printf("%s - %s\n", lvmname, cname->desc); else printf("%s\n", lvmname);