diff options
author | Richard Genoud <richard.genoud@posteo.net> | 2020-11-03 12:11:19 +0100 |
---|---|---|
committer | Tom Rini <trini@konsulko.com> | 2020-11-18 13:48:57 -0500 |
commit | 6756caec9c410567cddf0d93aac6f93f6d02abe4 (patch) | |
tree | 549ec9a1c8b5c0e16ba73adf6daaaea7bad065a7 | |
parent | 3b17c4164bfc3f68c89ad8120e772177e956135d (diff) | |
download | u-boot-6756caec9c410567cddf0d93aac6f93f6d02abe4.tar.gz |
fs/squashfs: sqfs_probe: fix possible memory leak on error
If SquashFS magic number is invalid, there's a memory leak.
Reviewed-by: Joao Marcos Costa <jmcosta944@gmail.com>
Signed-off-by: Richard Genoud <richard.genoud@posteo.net>
-rw-r--r-- | fs/squashfs/sqfs.c | 14 |
1 files changed, 9 insertions, 5 deletions
diff --git a/fs/squashfs/sqfs.c b/fs/squashfs/sqfs.c index d8d4584fbf..7d6f0e88e3 100644 --- a/fs/squashfs/sqfs.c +++ b/fs/squashfs/sqfs.c @@ -1090,8 +1090,8 @@ int sqfs_probe(struct blk_desc *fs_dev_desc, struct disk_partition *fs_partition /* Make sure it has a valid SquashFS magic number*/ if (get_unaligned_le32(&sblk->s_magic) != SQFS_MAGIC_NUMBER) { printf("Bad magic number for SquashFS image.\n"); - ctxt.cur_dev = NULL; - return -EINVAL; + ret = -EINVAL; + goto error; } ctxt.sblk = sblk; @@ -1099,12 +1099,16 @@ int sqfs_probe(struct blk_desc *fs_dev_desc, struct disk_partition *fs_partition ret = sqfs_decompressor_init(&ctxt); if (ret) { - ctxt.cur_dev = NULL; - free(ctxt.sblk); - return -EINVAL; + ret = -EINVAL; + goto error; } return 0; +error: + ctxt.cur_dev = NULL; + free(ctxt.sblk); + ctxt.sblk = NULL; + return ret; } static char *sqfs_basename(char *path) |