From: Richard Genoud <richard.genoud@posteo.net>
Date: Tue, 3 Nov 2020 11:11:13 +0000 (+0100)
Subject: fs/squashfs: sqfs_read: fix memory leak
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=b39504612a25b15bb2997889827166a2f24a86d6;p=platform%2Fkernel%2Fu-boot.git

fs/squashfs: sqfs_read: fix memory leak

sqfs_closedir() should be called to free memory allocated by
sqfs_opendir()

Reviewed-by: Joao Marcos Costa <jmcosta944@gmail.com>
Signed-off-by: Richard Genoud <richard.genoud@posteo.net>
[jh80.chung: cherry picked from mainline commit 7ce9745cbeae7733a7068f19a7fc3ec8f21dc38c]
Signed-off-by: Jaehoon Chung <jh80.chung@samsung.com>

Change-Id: I200f764bb6ce2bb828e2217d1b0f949e1db14597
---

diff --git a/fs/squashfs/sqfs.c b/fs/squashfs/sqfs.c
index 8e2863ac02..a71ada0660 100644
--- a/fs/squashfs/sqfs.c
+++ b/fs/squashfs/sqfs.c
@@ -1344,7 +1344,6 @@ int sqfs_read(const char *filename, void *buf, loff_t offset, loff_t len,
 	if (ret) {
 		printf("File not found.\n");
 		*actread = 0;
-		sqfs_closedir(dirsp);
 		ret = -ENOENT;
 		goto free_paths;
 	}
@@ -1535,6 +1534,7 @@ free_datablk:
 free_paths:
 	free(file);
 	free(dir);
+	sqfs_closedir(dirsp);
 
 	return ret;
 }