X-Git-Url: https://git.lttng.org/?p=lttng-tools.git;a=blobdiff_plain;f=src%2Fcommon%2Fnotification.c;h=2c4e052b1d0d14c577594a964e4e4d08a4eba022;hp=806c5561936a985142882636423c7e58c09c2269;hb=44635d77b591f83a80d48cd93497bd1cd6aa788d;hpb=c0a66c84b5b2484b75798aec7543b680b4d4ab6c

diff --git a/src/common/notification.c b/src/common/notification.c
index 806c55619..2c4e052b1 100644
--- a/src/common/notification.c
+++ b/src/common/notification.c
@@ -10,18 +10,19 @@
 #include <lttng/condition/evaluation-internal.h>
 #include <lttng/condition/condition.h>
 #include <lttng/condition/evaluation.h>
-#include <common/sessiond-comm/payload.h>
-#include <common/sessiond-comm/payload-view.h>
+#include <lttng/trigger/trigger-internal.h>
+#include <common/payload.h>
+#include <common/payload-view.h>
 #include <assert.h>
 
 LTTNG_HIDDEN
 struct lttng_notification *lttng_notification_create(
-		struct lttng_condition *condition,
+		struct lttng_trigger *trigger,
 		struct lttng_evaluation *evaluation)
 {
 	struct lttng_notification *notification = NULL;
 
-	if (!condition || !evaluation) {
+	if (!trigger || !evaluation) {
 		goto end;
 	}
 
@@ -30,7 +31,7 @@ struct lttng_notification *lttng_notification_create(
 		goto end;
 	}
 
-	notification->condition = condition;
+	notification->trigger = trigger;
 	notification->evaluation = evaluation;
 end:
 	return notification;
@@ -53,7 +54,7 @@ int lttng_notification_serialize(const struct lttng_notification *notification,
 	}
 
 	size_before_payload = payload->buffer.size;
-	ret = lttng_condition_serialize(notification->condition,
+	ret = lttng_trigger_serialize(notification->trigger,
 			payload);
 	if (ret) {
 		goto end;
@@ -78,16 +79,25 @@ ssize_t lttng_notification_create_from_payload(
 		struct lttng_notification **notification)
 {
 	ssize_t ret, notification_size = 0, condition_size, evaluation_size;
-	const struct lttng_notification_comm *notification_comm;
-	struct lttng_condition *condition;
+	struct lttng_trigger *trigger;
 	struct lttng_evaluation *evaluation;
+	const struct lttng_notification_comm *notification_comm;
+	const struct lttng_payload_view notification_comm_view =
+			lttng_payload_view_from_view(
+					src_view, 0, sizeof(*notification_comm));
 
 	if (!src_view || !notification) {
 		ret = -1;
 		goto end;
 	}
 
-	notification_comm = (typeof(notification_comm)) src_view->buffer.data;
+	if (!lttng_payload_view_is_valid(&notification_comm_view)) {
+		/* Payload not large enough to contain the header. */
+		ret = -1;
+		goto end;
+	}
+
+	notification_comm = (typeof(notification_comm)) notification_comm_view.buffer.data;
 	notification_size += sizeof(*notification_comm);
 	{
 		/* struct lttng_condition */
@@ -95,8 +105,8 @@ ssize_t lttng_notification_create_from_payload(
 				lttng_payload_view_from_view(src_view,
 						notification_size, -1);
 
-		condition_size = lttng_condition_create_from_payload(
-				&condition_view, &condition);
+		condition_size = lttng_trigger_create_from_payload(
+				&condition_view, &trigger);
 	}
 
 	if (condition_size < 0) {
@@ -113,6 +123,7 @@ ssize_t lttng_notification_create_from_payload(
 						notification_size, -1);
 
 		evaluation_size = lttng_evaluation_create_from_payload(
+				lttng_trigger_get_const_condition(trigger),
 				&evaluation_view, &evaluation);
 	}
 
@@ -130,7 +141,7 @@ ssize_t lttng_notification_create_from_payload(
 		goto error;
 	}
 
-	*notification = lttng_notification_create(condition, evaluation);
+	*notification = lttng_notification_create(trigger, evaluation);
 	if (!*notification) {
 		ret = -1;
 		goto error;
@@ -139,7 +150,7 @@ ssize_t lttng_notification_create_from_payload(
 end:
 	return ret;
 error:
-	lttng_condition_destroy(condition);
+	lttng_trigger_destroy(trigger);
 	lttng_evaluation_destroy(evaluation);
 	return ret;
 }
@@ -150,7 +161,7 @@ void lttng_notification_destroy(struct lttng_notification *notification)
 		return;
 	}
 
-	lttng_condition_destroy(notification->condition);
+	lttng_trigger_destroy(notification->trigger);
 	lttng_evaluation_destroy(notification->evaluation);
 	free(notification);
 }
@@ -158,7 +169,7 @@ void lttng_notification_destroy(struct lttng_notification *notification)
 const struct lttng_condition *lttng_notification_get_condition(
 		struct lttng_notification *notification)
 {
-	return notification ? notification->condition : NULL;
+	return notification ? lttng_trigger_get_const_condition(notification->trigger) : NULL;
 }
 
 const struct lttng_evaluation *lttng_notification_get_evaluation(
@@ -166,3 +177,9 @@ const struct lttng_evaluation *lttng_notification_get_evaluation(
 {
 	return notification ? notification->evaluation : NULL;
 }
+
+const struct lttng_trigger *lttng_notification_get_trigger(
+		struct lttng_notification *notification)
+{
+	return notification ? notification->trigger : NULL;
+}