diff options
| author | Stephen Warren <[email protected]> | 2015-09-04 22:03:45 -0600 |
|---|---|---|
| committer | Tom Rini <[email protected]> | 2015-09-11 17:15:23 -0400 |
| commit | d56b2015e6d5695d009d8e5e3930cbf34c3b5da0 (patch) | |
| tree | 1f0bd35221f5d2c16165f2d36e9d9f0ea706abdf | |
| parent | 934b14f2bb306dfa1947d30f4df90cdc8b710332 (diff) | |
ext4: fix leak in check_filename()
root_first_block_buffer should be free()d in all cases, not just when an
error occurs. Fix the success exit path of the function to do this.
Signed-off-by: Stephen Warren <[email protected]>
Acked-by: Lukasz Majewski <[email protected]>
Tested-by: Lukasz Majewski <[email protected]>
| -rw-r--r-- | fs/ext4/ext4_common.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/fs/ext4/ext4_common.c b/fs/ext4/ext4_common.c index 9fdbfe6e2db..410419e241b 100644 --- a/fs/ext4/ext4_common.c +++ b/fs/ext4/ext4_common.c @@ -770,6 +770,7 @@ static int check_filename(char *filename, unsigned int blknr) struct ext2_dirent *previous_dir = NULL; char *ptr = NULL; struct ext_filesystem *fs = get_fs(); + int ret = -1; /* get the first block of root */ first_block_no_of_root = blknr; @@ -823,12 +824,12 @@ static int check_filename(char *filename, unsigned int blknr) if (ext4fs_put_metadata(root_first_block_addr, first_block_no_of_root)) goto fail; - return inodeno; + ret = inodeno; } fail: free(root_first_block_buffer); - return -1; + return ret; } int ext4fs_filename_check(char *filename) |