diff options
author | Kay Sievers <kay.sievers@vrfy.org> | 2009-12-22 22:25:16 +0100 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@suse.de> | 2009-12-23 11:23:44 -0800 |
commit | 8042273801059884da2d53bbca34575d090b6f4e (patch) | |
tree | bc888394726172c0a489fd8bfa18b551272cbe69 | |
parent | 0787fdf70ba4c41a3350096ebaa347a17e900385 (diff) | |
download | op-kernel-dev-8042273801059884da2d53bbca34575d090b6f4e.zip op-kernel-dev-8042273801059884da2d53bbca34575d090b6f4e.tar.gz |
devtmpfs: unlock mutex in case of string allocation error
Reported-by: Kirill A. Shutemov <kirill@shutemov.name>
Signed-off-by: Kay Sievers <kay.sievers@vrfy.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
-rw-r--r-- | drivers/base/devtmpfs.c | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/drivers/base/devtmpfs.c b/drivers/base/devtmpfs.c index 278371c..090dd48 100644 --- a/drivers/base/devtmpfs.c +++ b/drivers/base/devtmpfs.c @@ -101,8 +101,10 @@ static int create_path(const char *nodepath) /* parent directories do not exist, create them */ path = kstrdup(nodepath, GFP_KERNEL); - if (!path) - return -ENOMEM; + if (!path) { + err = -ENOMEM; + goto out; + } s = path; for (;;) { s = strchr(s, '/'); @@ -117,6 +119,7 @@ static int create_path(const char *nodepath) } kfree(path); } +out: mutex_unlock(&dirlock); return err; } |