erofs-utils: fsck: Fix potential memory leak in error path

Free the memory allocated for 'entry' if strdup() fails.

Signed-off-by: Sandeep Dhavale <dhavale@google.com>
Link: https://lore.kernel.org/r/20230913221104.429825-2-dhavale@google.com
Signed-off-by: Gao Xiang <hsiangkao@linux.alibaba.com>

diff --git a/fsck/main.c b/fsck/main.c
index 7f78513c385e8e13793118be006ec160d5047068..3f86da4bdd206bfb521bf71cd9d062558aa42892 100644 (file)
--- a/fsck/main.c
+++ b/fsck/main.c
@@ -585,8 +585,10 @@ static int erofsfsck_hardlink_insert(erofs_nid_t nid, const char *path)
 
        entry->nid = nid;
        entry->path = strdup(path);
-       if (!entry->path)
+       if (!entry->path) {
+               free(entry);
                return -ENOMEM;
+       }
 
        list_add_tail(&entry->list,
                      &erofsfsck_link_hashtable[nid % NR_HARDLINK_HASHTABLE]);