Commit c24e9b3f authored by Cyrill Gorcunov's avatar Cyrill Gorcunov Committed by Linus Torvalds

capifs: fix memory leak on remount

capifs_remount may reach 'return' statement without freeing of memory
allocated by kstrdup call
Signed-off-by: default avatarCyrill Gorcunov <gorcunov@gmail.com>
Cc: Karsten Keil <kkeil@suse.de>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
parent 30d55e71
...@@ -69,6 +69,7 @@ static int capifs_remount(struct super_block *s, int *flags, char *data) ...@@ -69,6 +69,7 @@ static int capifs_remount(struct super_block *s, int *flags, char *data)
} else if (sscanf(this_char, "mode=%o%c", &n, &dummy) == 1) } else if (sscanf(this_char, "mode=%o%c", &n, &dummy) == 1)
mode = n & ~S_IFMT; mode = n & ~S_IFMT;
else { else {
kfree(new_opt);
printk("capifs: called with bogus options\n"); printk("capifs: called with bogus options\n");
return -EINVAL; return -EINVAL;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment