]> granicus.if.org Git - sudo/commitdiff
In fill_args(), check for "arg_size == 0" instead of
authorTodd C. Miller <Todd.Miller@courtesan.com>
Thu, 5 May 2016 21:12:37 +0000 (15:12 -0600)
committerTodd C. Miller <Todd.Miller@courtesan.com>
Thu, 5 May 2016 21:12:37 +0000 (15:12 -0600)
"sudoerslval.command.args == NULL" since the latter leads Coverity
to imply that sudoerslval.command.args could be NULL later on.
Coverity CID 104093.

plugins/sudoers/toke_util.c

index be46782f966f9c7c96e23092f113f94237323c3b..01b4fe1106ff41e8f9c556412b7b75bbcb6882ff 100644 (file)
@@ -131,7 +131,7 @@ fill_args(const char *s, size_t len, int addspace)
     char *p;
     debug_decl(fill_args, SUDOERS_DEBUG_PARSER)
 
-    if (sudoerslval.command.args == NULL) {
+    if (arg_size == 0) {
        addspace = 0;
        new_len = len;
     } else