Commit deae6cfc authored by Marcin Slusarz's avatar Marcin Slusarz Committed by Linus Torvalds

udf: check if udf_load_logicalvol failed

udf_load_logicalvol may fail eg in out of memory conditions - check it
and propagate error further
Signed-off-by: default avatarMarcin Slusarz <marcin.slusarz@gmail.com>
Cc: Ben Fennema <bfennema@falcon.csc.calpoly.edu>
Acked-by: default avatarJan Kara <jack@suse.cz>
Cc: Christoph Hellwig <hch@infradead.org>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
parent dc5d39be
...@@ -1263,7 +1263,10 @@ static int udf_process_sequence(struct super_block *sb, long block, ...@@ -1263,7 +1263,10 @@ static int udf_process_sequence(struct super_block *sb, long block,
if (i == VDS_POS_PRIMARY_VOL_DESC) { if (i == VDS_POS_PRIMARY_VOL_DESC) {
udf_load_pvoldesc(sb, bh); udf_load_pvoldesc(sb, bh);
} else if (i == VDS_POS_LOGICAL_VOL_DESC) { } else if (i == VDS_POS_LOGICAL_VOL_DESC) {
udf_load_logicalvol(sb, bh, fileset); /* TODO: check return value */ if (udf_load_logicalvol(sb, bh, fileset)) {
brelse(bh);
return 1;
}
} else if (i == VDS_POS_PARTITION_DESC) { } else if (i == VDS_POS_PARTITION_DESC) {
struct buffer_head *bh2 = NULL; struct buffer_head *bh2 = NULL;
if (udf_load_partdesc(sb, bh)) { if (udf_load_partdesc(sb, bh)) {
......
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