Fix: tests: fix unused-but-set warning in test_fd_tracker.c

[lttng-tools.git] / src / bin / lttng-sessiond / trigger-error-query.c

diff --git a/src/bin/lttng-sessiond/trigger-error-query.c b/src/bin/lttng-sessiond/trigger-error-query.c
index 4c464004d52f49fffcc1769fc8a338926ff0ed62..8c39e1b0655326bf83b587877c47fa17bf7f4d82 100644 (file)
--- a/src/bin/lttng-sessiond/trigger-error-query.c
+++ b/src/bin/lttng-sessiond/trigger-error-query.c
@@ -10,10 +10,16 @@
 #include <lttng/trigger/trigger-internal.h>
 #include <lttng/action/action-internal.h>
 
-LTTNG_HIDDEN
 enum lttng_trigger_status lttng_trigger_add_error_results(
                const struct lttng_trigger *trigger,
                struct lttng_error_query_results *results)
+{
+       return LTTNG_TRIGGER_STATUS_OK;
+}
+
+enum lttng_trigger_status lttng_trigger_condition_add_error_results(
+               const struct lttng_trigger *trigger,
+               struct lttng_error_query_results *results)
 {
        enum lttng_trigger_status status;
        uint64_t discarded_tracer_messages_count;
@@ -27,9 +33,14 @@ enum lttng_trigger_status lttng_trigger_add_error_results(
                        trigger_name : "(anonymous)";
        status = lttng_trigger_get_owner_uid(trigger,
                        &trigger_owner);
-       assert(status == LTTNG_TRIGGER_STATUS_OK);
+       LTTNG_ASSERT(status == LTTNG_TRIGGER_STATUS_OK);
 
-       /* Only add discarded tracer messages count for applicable triggers. */
+       /*
+        * Only add discarded tracer messages count for applicable conditions.
+        * As of 2.13, only "event rule matches" conditions can generate
+        * reportable errors hence why this function is very specific to this
+        * condition type.
+        */
        if (!lttng_trigger_needs_tracer_notifier(trigger)) {
                status = LTTNG_TRIGGER_STATUS_OK;
                goto end;
@@ -38,7 +49,7 @@ enum lttng_trigger_status lttng_trigger_add_error_results(
        error_accounting_status = event_notifier_error_accounting_get_count(
                        trigger, &discarded_tracer_messages_count);
        if (error_accounting_status != EVENT_NOTIFIER_ERROR_ACCOUNTING_STATUS_OK) {
-               ERR("Failed to retrieve tracer discarded messages count for triger: triggger name = '%s', trigger owner uid = %d",
+               ERR("Failed to retrieve tracer discarded messages count for trigger: trigger name = '%s', trigger owner uid = %d",
                                trigger_name, (int) trigger_owner);
                status = LTTNG_TRIGGER_STATUS_ERROR;
                goto end;
@@ -68,7 +79,6 @@ end:
        return status;
 }
 
-LTTNG_HIDDEN
 enum lttng_trigger_status lttng_trigger_add_action_error_query_results(
                struct lttng_trigger *trigger,
                struct lttng_error_query_results *results)
@@ -83,7 +93,7 @@ enum lttng_trigger_status lttng_trigger_add_action_error_query_results(
                        trigger_name : "(anonymous)";
        status = lttng_trigger_get_owner_uid(trigger,
                        &trigger_owner);
-       assert(status == LTTNG_TRIGGER_STATUS_OK);
+       LTTNG_ASSERT(status == LTTNG_TRIGGER_STATUS_OK);
 
        action_status = lttng_action_add_error_query_results(
                        lttng_trigger_get_action(trigger), results);