Fix: lttng_set_event_filter() was returning bad error code

[lttng-tools.git] / src / bin / lttng / commands / enable_events.c

diff --git a/src/bin/lttng/commands/enable_events.c b/src/bin/lttng/commands/enable_events.c
index 9a4b9aac90c50753b62524404a487dd0578c61cf..0bd96eb303680cc715dced1bda4eb845d74c75eb 100644 (file)
--- a/src/bin/lttng/commands/enable_events.c
+++ b/src/bin/lttng/commands/enable_events.c
@@ -422,10 +422,13 @@ static int enable_events(char *session_name)
                        if (ret < 0) {
                                switch (-ret) {
                                case LTTNG_ERR_FILTER_EXIST:
-                                       ERR("Filter on events is already enabled"
+                                       WARN("Filter on events is already enabled"
                                                        " (channel %s, session %s)",
                                                channel_name, session_name);
                                        break;
+                               case LTTNG_ERR_FILTER_INVAL:
+                               case LTTNG_ERR_FILTER_NOMEM:
+                                       ERR("%s", lttng_strerror(ret));
                                default:
                                        ERR("Setting filter: '%s'", opt_filter);
                                        /*
@@ -579,10 +582,13 @@ static int enable_events(char *session_name)
                        if (ret < 0) {
                                switch (-ret) {
                                case LTTNG_ERR_FILTER_EXIST:
-                                       ERR("Filter on event %s is already enabled"
+                                       WARN("Filter on event %s is already enabled"
                                                        " (channel %s, session %s)",
                                                event_name, channel_name, session_name);
                                        break;
+                               case LTTNG_ERR_FILTER_INVAL:
+                               case LTTNG_ERR_FILTER_NOMEM:
+                                       ERR("%s", lttng_strerror(ret));
                                default:
                                        ERR("Setting filter for event %s: '%s'", ev.name,
                                                        opt_filter);