X-Git-Url: http://git.liburcu.org/?a=blobdiff_plain;f=src%2Fbin%2Flttng-sessiond%2Fcmd.c;h=6259ed089baa5f401990e90cb5034b3a048f1109;hb=0a3fca4d597b02143affd90d50aa33409709024a;hp=63d91f53d197ecc4b54d19a00dafee24668011f5;hpb=79e71bf5a317ec14b1e02fe35ccf2ad52c14883c;p=lttng-tools.git

diff --git a/src/bin/lttng-sessiond/cmd.c b/src/bin/lttng-sessiond/cmd.c
index 63d91f53d..6259ed089 100644
--- a/src/bin/lttng-sessiond/cmd.c
+++ b/src/bin/lttng-sessiond/cmd.c
@@ -1737,7 +1737,8 @@ static int _cmd_enable_event(struct ltt_session *session,
 		memset(&uevent, 0, sizeof(uevent));
 		uevent.type = LTTNG_EVENT_TRACEPOINT;
 		uevent.loglevel_type = LTTNG_EVENT_LOGLEVEL_ALL;
-		default_event_name = event_get_default_agent_ust_name(domain->type);
+		default_event_name = event_get_default_agent_ust_name(
+				domain->type);
 		if (!default_event_name) {
 			ret = LTTNG_ERR_FATAL;
 			goto error;
@@ -1769,29 +1770,30 @@ static int _cmd_enable_event(struct ltt_session *session,
 		}
 
 		{
-			struct lttng_filter_bytecode *filter_copy = NULL;
 			char *filter_expression_copy = NULL;
+			struct lttng_filter_bytecode *filter_copy = NULL;
 
 			if (filter) {
-				filter_copy = zmalloc(
-					sizeof(struct lttng_filter_bytecode)
-					+ filter->len);
+				const size_t filter_size = sizeof(
+						struct lttng_filter_bytecode)
+						+ filter->len;
+
+				filter_copy = zmalloc(filter_size);
 				if (!filter_copy) {
 					ret = LTTNG_ERR_NOMEM;
-					goto error;
 				}
+				memcpy(filter_copy, filter, filter_size);
 
-				memcpy(filter_copy, filter,
-				       sizeof(struct lttng_filter_bytecode)
-				       + filter->len);
-			}
-
-			if (filter_expression) {
 				filter_expression_copy =
 						strdup(filter_expression);
 				if (!filter_expression) {
 					ret = LTTNG_ERR_NOMEM;
-					goto error_free_copy;
+				}
+
+				if (!filter_expression_copy || !filter_copy) {
+					free(filter_expression_copy);
+					free(filter_copy);
+					goto error;
 				}
 			}
 
@@ -1799,11 +1801,6 @@ static int _cmd_enable_event(struct ltt_session *session,
 					(char *) default_chan_name,
 					&uevent, filter_expression_copy,
 					filter_copy, NULL, wpipe);
-			filter_copy = NULL;
-			filter_expression_copy = NULL;
-error_free_copy:
-			free(filter_copy);
-			free(filter_expression_copy);
 		}
 
 		if (ret != LTTNG_OK && ret != LTTNG_ERR_UST_EVENT_ENABLED) {
@@ -1814,12 +1811,12 @@ error_free_copy:
 		if (strncmp(event->name, "*", 1) == 0 && strlen(event->name) == 1) {
 			ret = event_agent_enable_all(usess, agt, event, filter,
 					filter_expression);
-			filter = NULL;
 		} else {
 			ret = event_agent_enable(usess, agt, event, filter,
 					filter_expression);
-			filter = NULL;
 		}
+		filter = NULL;
+		filter_expression = NULL;
 		if (ret != LTTNG_OK) {
 			goto error;
 		}