From: David Goulet <dgoulet@efficios.com>
Date: Tue, 19 Jun 2012 15:15:38 +0000 (-0400)
Subject: Fix: unchecked pointer from getenv() for lttng create
X-Git-Tag: v2.1.0-rc1~107
X-Git-Url: https://git.lttng.org./?a=commitdiff_plain;h=d386b2341d96bbeeb1f89071614506c0ddbb9bcc;p=lttng-tools.git

Fix: unchecked pointer from getenv() for lttng create

The lttng create command was not handling the return pointer from
getenv("HOME") and was passing it to a strdup() call.

(close #271)

Signed-off-by: David Goulet <dgoulet@efficios.com>
---

diff --git a/src/bin/lttng/commands/create.c b/src/bin/lttng/commands/create.c
index 2778ef553..eac261ac7 100644
--- a/src/bin/lttng/commands/create.c
+++ b/src/bin/lttng/commands/create.c
@@ -94,13 +94,14 @@ static int create_session()
 
 	/* Auto output path */
 	if (opt_output_path == NULL) {
-		alloc_path = strdup(config_get_default_path());
+		alloc_path = config_get_default_path();
 		if (alloc_path == NULL) {
-			ERR("Home path not found.\n \
+			ERR("HOME path not found.\n \
 				 Please specify an output path using -o, --output PATH");
 			ret = CMD_FATAL;
 			goto error;
 		}
+		alloc_path = strdup(alloc_path);
 
 		if (have_name) {
 			ret = asprintf(&traces_path, "%s/" DEFAULT_TRACE_DIR_NAME