Commit 1b1baff6 authored by Namjae Jeon's avatar Namjae Jeon Committed by Jan Kara

UDF: Fix a null pointer dereference in udf_sb_free_partitions

This patch fixes a regression caused by commit bff943af "udf: Fix memory
leak when mounting" due to which it was triggering a kernel null point
dereference in case of interrupted mount OR when allocating memory to
sbi->s_partmaps failed in function udf_sb_alloc_partition_maps.
Reported-and-tested-by: default avatarJames Hogan <james@albanarts.com>
Signed-off-by: default avatarNamjae Jeon <namjae.jeon@samsung.com>
Signed-off-by: default avatarAshish Sangwan <a.sangwan@samsung.com>
Signed-off-by: default avatarJan Kara <jack@suse.cz>
parent 7e2fb2d7
...@@ -307,7 +307,8 @@ static void udf_sb_free_partitions(struct super_block *sb) ...@@ -307,7 +307,8 @@ static void udf_sb_free_partitions(struct super_block *sb)
{ {
struct udf_sb_info *sbi = UDF_SB(sb); struct udf_sb_info *sbi = UDF_SB(sb);
int i; int i;
if (sbi->s_partmaps == NULL)
return;
for (i = 0; i < sbi->s_partitions; i++) for (i = 0; i < sbi->s_partitions; i++)
udf_free_partition(&sbi->s_partmaps[i]); udf_free_partition(&sbi->s_partmaps[i]);
kfree(sbi->s_partmaps); kfree(sbi->s_partmaps);
......
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