Fix: add_trigger.c: `goto error` with a wrong UID for `--owner-uid`

Signed-off-by: Philippe Proulx <eeppeliteloop@gmail.com>
Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>
Change-Id: I8650af1f43e8ed8bad58a928cfcdf78d382212f1

diff --git a/src/bin/lttng/commands/add_trigger.c b/src/bin/lttng/commands/add_trigger.c
index 9c6b2cceb7714428092697b881d49a42241e0671..ac086891042d41af4c836f62b335e3a39ee78a7c 100644 (file)
--- a/src/bin/lttng/commands/add_trigger.c
+++ b/src/bin/lttng/commands/add_trigger.c
@@ -2200,6 +2200,7 @@ int cmd_add_trigger(int argc, const char **argv)
                uid = strtol(owner_uid, &end, 10);
                if (end == owner_uid || *end != '\0' || errno != 0) {
                        ERR("Failed to parse `%s` as a user id.", owner_uid);
                uid = strtol(owner_uid, &end, 10);
                if (end == owner_uid || *end != '\0' || errno != 0) {
                        ERR("Failed to parse `%s` as a user id.", owner_uid);

+                       goto error;

                }
 
                trigger_status = lttng_trigger_set_owner_uid(trigger, uid);
                }
 
                trigger_status = lttng_trigger_set_owner_uid(trigger, uid);