[lvm-devel] [PATCH 1/4] Fix coredump and memory leak for 'dmsetup help -c'

Zdenek Kabelac zkabelac at redhat.com
Thu Dec 10 12:53:56 UTC 2009


Signed-off-by: Zdenek Kabelac <zkabelac at redhat.com>
---
 tools/dmsetup.c |    6 ++++--
 1 files changed, 4 insertions(+), 2 deletions(-)

diff --git a/tools/dmsetup.c b/tools/dmsetup.c
index 888edff..b5568f0 100644
--- a/tools/dmsetup.c
+++ b/tools/dmsetup.c
@@ -2402,7 +2402,7 @@ static int _report_init(struct command *c)
 	size_t len = 0;
 	int r = 0;
 
-	if (!strcmp(c->name, "splitname"))
+	if (c && !strcmp(c->name, "splitname"))
 		options = (char *) splitname_report_options;
 
 	/* emulate old dmsetup behaviour */
@@ -2591,7 +2591,9 @@ static int _help(int argc __attribute((unused)),
 		_switches[OPTIONS_ARG] = 1;
 		_string_args[OPTIONS_ARG] = (char *) "help";
 		_switches[SORT_ARG] = 0;
-	
+
+		if (_report)
+			dm_report_free(_report);
 		(void) _report_init(NULL);
 	}
 
-- 
1.6.5.3




More information about the lvm-devel mailing list