X-Git-Url: https://git.lttng.org/?p=lttng-tools.git;a=blobdiff_plain;f=src%2Fcommon%2Fsession-consumed-size.c;h=6ba64249d8cf77566eb4c0be9a072100df4c9403;hp=f29176f8df91e08ca166d74a04f1ff0f12f2ae6c;hb=9b63a4aa352c184984395c097f86d5e54c295012;hpb=3647288fe42b25340f905046f3bd9aef21e12aaa

diff --git a/src/common/session-consumed-size.c b/src/common/session-consumed-size.c
index f29176f8d..6ba64249d 100644
--- a/src/common/session-consumed-size.c
+++ b/src/common/session-consumed-size.c
@@ -59,11 +59,11 @@ bool lttng_condition_session_consumed_size_validate(
 	consumed = container_of(condition, struct lttng_condition_session_consumed_size,
 			parent);
 	if (!consumed->session_name) {
-		ERR("Invalid buffer condition: a target session name must be set.");
+		ERR("Invalid session consumed size condition: a target session name must be set.");
 		goto end;
 	}
 	if (!consumed->consumed_threshold_bytes.set) {
-		ERR("Invalid session condition: a threshold must be set.");
+		ERR("Invalid session consumed size condition: a threshold must be set.");
 		goto end;
 	}
 
@@ -136,8 +136,9 @@ bool lttng_condition_session_consumed_size_is_equal(const struct lttng_condition
 		}
 	}
 
-	if ((a->session_name && !b->session_name) ||
-			(!a->session_name && b->session_name)) {
+	assert(a->session_name);
+	assert(b->session_name);
+	if (strcmp(a->session_name, b->session_name)) {
 		goto end;
 	}
 
@@ -198,7 +199,7 @@ ssize_t init_condition_from_buffer(struct lttng_condition *condition,
 	status = lttng_condition_session_consumed_size_set_threshold(condition,
 			condition_comm->consumed_threshold_bytes);
 	if (status != LTTNG_CONDITION_STATUS_OK) {
-		ERR("Failed to initialize session consumed condition threshold");
+		ERR("Failed to initialize session consumed size condition threshold");
 		ret = -1;
 		goto end;
 	}
@@ -213,7 +214,7 @@ ssize_t init_condition_from_buffer(struct lttng_condition *condition,
 	status = lttng_condition_session_consumed_size_set_session_name(condition,
 			session_name);
 	if (status != LTTNG_CONDITION_STATUS_OK) {
-		ERR("Failed to set buffer consumed session name");
+		ERR("Failed to set session consumed size condition's session name");
 		ret = -1;
 		goto end;
 	}
@@ -258,7 +259,6 @@ error:
 
 static
 struct lttng_evaluation *create_evaluation_from_buffer(
-		enum lttng_condition_type type,
 		const struct lttng_buffer_view *view)
 {
 	const struct lttng_evaluation_session_consumed_size_comm *comm =
@@ -269,7 +269,7 @@ struct lttng_evaluation *create_evaluation_from_buffer(
 		goto end;
 	}
 
-	evaluation = lttng_evaluation_session_consumed_size_create(type,
+	evaluation = lttng_evaluation_session_consumed_size_create(
 			comm->session_consumed);
 end:
 	return evaluation;
@@ -288,8 +288,7 @@ ssize_t lttng_evaluation_session_consumed_size_create_from_buffer(
 		goto error;
 	}
 
-	evaluation = create_evaluation_from_buffer(
-			LTTNG_CONDITION_TYPE_SESSION_CONSUMED_SIZE, view);
+	evaluation = create_evaluation_from_buffer(view);
 	if (!evaluation) {
 		ret = -1;
 		goto error;
@@ -403,7 +402,7 @@ end:
 
 static
 int lttng_evaluation_session_consumed_size_serialize(
-		struct lttng_evaluation *evaluation,
+		const struct lttng_evaluation *evaluation,
 		struct lttng_dynamic_buffer *buf)
 {
 	struct lttng_evaluation_session_consumed_size *consumed;
@@ -428,7 +427,7 @@ void lttng_evaluation_session_consumed_size_destroy(
 
 LTTNG_HIDDEN
 struct lttng_evaluation *lttng_evaluation_session_consumed_size_create(
-		enum lttng_condition_type type, uint64_t consumed)
+		uint64_t consumed)
 {
 	struct lttng_evaluation_session_consumed_size *consumed_eval;
 
@@ -437,7 +436,7 @@ struct lttng_evaluation *lttng_evaluation_session_consumed_size_create(
 		goto end;
 	}
 
-	consumed_eval->parent.type = type;
+	consumed_eval->parent.type = LTTNG_CONDITION_TYPE_SESSION_CONSUMED_SIZE;
 	consumed_eval->session_consumed = consumed;
 	consumed_eval->parent.serialize = lttng_evaluation_session_consumed_size_serialize;
 	consumed_eval->parent.destroy = lttng_evaluation_session_consumed_size_destroy;