devtmpfs: missing initialialization in never-hit case
authorAl Viro <viro@zeniv.linux.org.uk>
Thu, 28 Jul 2011 02:27:33 +0000 (22:27 -0400)
committerAl Viro <viro@zeniv.linux.org.uk>
Thu, 28 Jul 2011 02:27:33 +0000 (22:27 -0400)
create_path() on something without a single / in it will return err
without initializing it.  It actually can't happen (we call that thing
only if create on the same path returns -ENOENT, which won't happen
happen for single-component path), but in this case initializing err
to 0 is more than making compiler to STFU - would be the right thing
to return on such paths; the function creates a parent directory of
given pathname and in that case it has no work to do...

Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
drivers/base/devtmpfs.c

index b89fffc..33e1bed 100644 (file)
@@ -166,7 +166,7 @@ static int create_path(const char *nodepath)
 {
        char *path;
        char *s;
 {
        char *path;
        char *s;
-       int err;
+       int err = 0;
 
        /* parent directories do not exist, create them */
        path = kstrdup(nodepath, GFP_KERNEL);
 
        /* parent directories do not exist, create them */
        path = kstrdup(nodepath, GFP_KERNEL);