Commit 4f34a513 authored by Chengguang Xu's avatar Chengguang Xu Committed by Jan Kara

isofs: fix potential memory leak in mount option parsing

When specifying string type mount option (e.g., iocharset)
several times in a mount, current option parsing may
cause memory leak. Hence, call kfree for previous one
in this case. Meanwhile, check memory allocation result
for it.
Signed-off-by: default avatarChengguang Xu <cgxu519@gmx.com>
Signed-off-by: default avatarJan Kara <jack@suse.cz>
parent 96348e49
...@@ -394,7 +394,10 @@ static int parse_options(char *options, struct iso9660_options *popt) ...@@ -394,7 +394,10 @@ static int parse_options(char *options, struct iso9660_options *popt)
break; break;
#ifdef CONFIG_JOLIET #ifdef CONFIG_JOLIET
case Opt_iocharset: case Opt_iocharset:
kfree(popt->iocharset);
popt->iocharset = match_strdup(&args[0]); popt->iocharset = match_strdup(&args[0]);
if (!popt->iocharset)
return 0;
break; break;
#endif #endif
case Opt_map_a: case Opt_map_a:
......
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