LU-12641 libext2fs: memory leak of check_if_lustre_mounted

check_if_lustre_mounted() doesn't free the allocated memory
if failure.

Change-Id: I36b5c1e981ca4f1db8c9515be29dd98c074d14dc
Signed-off-by: Li Xi <lixi@ddn.com>
Reviewed-on: https://review.whamcloud.com/35729
Reviewed-by: Andreas Dilger <adilger@whamcloud.com>
Tested-by: jenkins <devops@whamcloud.com>
Reviewed-by: Wang Shilong <wshilong@whamcloud.com>
Tested-by: Maloo <maloo@whamcloud.com>

diff --git a/lib/ext2fs/ismounted.c b/lib/ext2fs/ismounted.c
index 0d7c5e2..0a78497 100644 (file)
--- a/lib/ext2fs/ismounted.c
+++ b/lib/ext2fs/ismounted.c
@@ -510,7 +510,8 @@ static errcode_t check_if_lustre_mounted(const char *device, int *mount_flags)
 
        if (realpath(device, real_device) == NULL) {
                fprintf(stderr, "Cannot resolve path %s\n", device);
-               return EXT2_ET_BAD_DEVICE_NAME;
+               rc = EXT2_ET_BAD_DEVICE_NAME;
+               goto out_free;
        }
 
        rc = check_lustre_proc_vals("/proc/fs/lustre/osd-ldiskfs", real_device);
@@ -520,9 +521,10 @@ static errcode_t check_if_lustre_mounted(const char *device, int *mount_flags)
        if (rc)
                *mount_flags |= EXT2_MF_MOUNTED;
 
+out_free:
        free(real_device);
 
-       return 0;
+       return rc;
 }