summaryrefslogtreecommitdiffstats
path: root/drivers/base
diff options
context:
space:
mode:
authorAl Viro <viro@zeniv.linux.org.uk>2011-07-27 22:27:33 -0400
committerAl Viro <viro@zeniv.linux.org.uk>2011-07-27 22:27:33 -0400
commit9d108d25487bf958f8093409a4c0bee6169edba6 (patch)
tree14e9e96403ffcec690f77f9e6f99429a1a2e535c /drivers/base
parentd6b722aa383a467a43d09ee38e866981abba08ab (diff)
downloadop-kernel-dev-9d108d25487bf958f8093409a4c0bee6169edba6.zip
op-kernel-dev-9d108d25487bf958f8093409a4c0bee6169edba6.tar.gz
devtmpfs: missing initialialization in never-hit case
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>
Diffstat (limited to 'drivers/base')
-rw-r--r--drivers/base/devtmpfs.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/base/devtmpfs.c b/drivers/base/devtmpfs.c
index b89fffc..33e1bed 100644
--- a/drivers/base/devtmpfs.c
+++ b/drivers/base/devtmpfs.c
@@ -166,7 +166,7 @@ static int create_path(const char *nodepath)
{
char *path;
char *s;
- int err;
+ int err = 0;
/* parent directories do not exist, create them */
path = kstrdup(nodepath, GFP_KERNEL);
OpenPOWER on IntegriCloud