aboutsummaryrefslogtreecommitdiff
path: root/fs
diff options
context:
space:
mode:
authorStefan Roese <sr@denx.de>2012-08-28 14:00:24 +0200
committerStefan Roese <sr@denx.de>2012-09-03 11:37:50 +0200
commitbe73913b91ca1e86c7478c1ab8af7b7ba6f3e4ad (patch)
treedaeb0b548bb03d14de96eecb410fefb9974d94af /fs
parenta005f19eff946454985785788c344f10616d571e (diff)
downloadu-boot-linaro-stable-be73913b91ca1e86c7478c1ab8af7b7ba6f3e4ad.tar.gz
ubifs: Fix memory leak in ubifs_finddir
This patch fixes a memory leak in ubifs_finddir(). Signed-off-by: Stefan Roese <sr@denx.de> Cc: dev.ma.dma@gmail.com
Diffstat (limited to 'fs')
-rw-r--r--fs/ubifs/ubifs.c11
1 files changed, 6 insertions, 5 deletions
diff --git a/fs/ubifs/ubifs.c b/fs/ubifs/ubifs.c
index 604eb8fdc..c68802bc8 100644
--- a/fs/ubifs/ubifs.c
+++ b/fs/ubifs/ubifs.c
@@ -295,6 +295,7 @@ static int ubifs_finddir(struct super_block *sb, char *dirname,
struct file *file;
struct dentry *dentry;
struct inode *dir;
+ int ret = 0;
file = kzalloc(sizeof(struct file), 0);
dentry = kzalloc(sizeof(struct dentry), 0);
@@ -336,7 +337,8 @@ static int ubifs_finddir(struct super_block *sb, char *dirname,
if ((strncmp(dirname, (char *)dent->name, nm.len) == 0) &&
(strlen(dirname) == nm.len)) {
*inum = le64_to_cpu(dent->inum);
- return 1;
+ ret = 1;
+ goto out_free;
}
/* Switch to the next entry */
@@ -355,11 +357,10 @@ static int ubifs_finddir(struct super_block *sb, char *dirname,
}
out:
- if (err != -ENOENT) {
+ if (err != -ENOENT)
ubifs_err("cannot find next direntry, error %d", err);
- return err;
- }
+out_free:
if (file->private_data)
kfree(file->private_data);
if (file)
@@ -369,7 +370,7 @@ out:
if (dir)
free(dir);
- return 0;
+ return ret;
}
static unsigned long ubifs_findfile(struct super_block *sb, char *filename)