Commit e930d420 authored by Dave Jones's avatar Dave Jones Committed by Linus Torvalds

[PATCH] Fix leaks in ISOFS.

Spotted with the source checker from Coverity.com.
Signed-off-by: default avatarDave Jones <davej@redhat.com>
Signed-off-by: default avatarLinus Torvalds <torvalds@osdl.org>
parent 2327d2ee
...@@ -153,6 +153,7 @@ int get_rock_ridge_filename(struct iso_directory_record * de, ...@@ -153,6 +153,7 @@ int get_rock_ridge_filename(struct iso_directory_record * de,
} }
} }
MAYBE_CONTINUE(repeat,inode); MAYBE_CONTINUE(repeat,inode);
if (buffer) kfree(buffer);
return retnamlen; /* If 0, this file did not have a NM field */ return retnamlen; /* If 0, this file did not have a NM field */
out: out:
if(buffer) kfree(buffer); if(buffer) kfree(buffer);
...@@ -351,7 +352,6 @@ int parse_rock_ridge_inode_internal(struct iso_directory_record * de, ...@@ -351,7 +352,6 @@ int parse_rock_ridge_inode_internal(struct iso_directory_record * de,
} }
} }
MAYBE_CONTINUE(repeat,inode); MAYBE_CONTINUE(repeat,inode);
return 0;
out: out:
if(buffer) kfree(buffer); if(buffer) kfree(buffer);
return 0; return 0;
...@@ -515,6 +515,8 @@ static int rock_ridge_symlink_readpage(struct file *file, struct page *page) ...@@ -515,6 +515,8 @@ static int rock_ridge_symlink_readpage(struct file *file, struct page *page)
} }
} }
MAYBE_CONTINUE(repeat, inode); MAYBE_CONTINUE(repeat, inode);
if (buffer)
kfree(buffer);
if (rpnt == link) if (rpnt == link)
goto fail; goto fail;
......
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