]> Pileus Git - ~andy/linux/commitdiff
params.c: fix Smack complaint about parse_args
authorJim Cromie <jim.cromie@gmail.com>
Thu, 3 May 2012 17:57:37 +0000 (11:57 -0600)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Sat, 5 May 2012 00:24:22 +0000 (17:24 -0700)
In commit 9fb48c744: "params: add 3rd arg to option handler callback
signature", the if-guard added to the pr_debug was overzealous; no
callers pass NULL, and existing code above and below the guard assumes
as much.  Change the if-guard to match, and silence the Smack
complaint.

CC: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Jim Cromie <jim.cromie@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
kernel/params.c

index b60e2c74b96121982240127245f2c9127cf4c2dc..be78c904b5646a47d0a2ea5840bb1dbd35fce0f0 100644 (file)
@@ -190,7 +190,7 @@ int parse_args(const char *doing,
        /* Chew leading spaces */
        args = skip_spaces(args);
 
-       if (args && *args)
+       if (*args)
                pr_debug("doing %s, parsing ARGS: '%s'\n", doing, args);
 
        while (*args) {