Whamcloud - gitweb
LU-6543 utils: fix memory leak in lfs_setstripe 09/14809/2
authorDmitry Eremin <dmitry.eremin@intel.com>
Thu, 14 May 2015 16:15:19 +0000 (19:15 +0300)
committerOleg Drokin <oleg.drokin@intel.com>
Fri, 10 Jul 2015 03:08:07 +0000 (03:08 +0000)
Allocated buffer param should be freed before return.

Signed-off-by: Dmitry Eremin <dmitry.eremin@intel.com>
Change-Id: I22d21fb12c0f62330a7c2ca6296f571db71084f2
Reviewed-on: http://review.whamcloud.com/14809
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Bob Glossman <bob.glossman@intel.com>
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Reviewed-by: John L. Hammond <john.hammond@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
lustre/utils/lfs.c

index f8ec6d5..c8d5dc4 100644 (file)
@@ -1050,6 +1050,7 @@ static int lfs_setstripe(int argc, char **argv)
                        fprintf(stderr, "error: %s: stripe count '%d' doesn't "
                                "match the number of OSTs: %d\n",
                                argv[0], st_count, nr_osts);
+                       free(param);
                        return CMD_HELP;
                }