enum {
OPT_HELP,
OPT_LIST_OPTIONS,
- OPT_USER_ID,
+ OPT_OWNER_UID,
};
static const
struct argpar_opt_descr remove_trigger_options[] = {
{ OPT_HELP, 'h', "help", false },
{ OPT_LIST_OPTIONS, '\0', "list-options", false },
- { OPT_USER_ID, '\0', "user-id", true },
+ { OPT_OWNER_UID, '\0', "owner-uid", true },
ARGPAR_OPT_DESCR_SENTINEL,
};
unsigned int triggers_count;
enum lttng_trigger_status trigger_status;
const struct lttng_trigger *trigger_to_remove = NULL;
- char *user_id = NULL;
+ char *owner_uid = NULL;
long long uid;
argpar_parse_ret = argpar_parse(argc - 1, argv + 1,
remove_trigger_options);
ret = 0;
goto end;
- case OPT_USER_ID:
+ case OPT_OWNER_UID:
{
- if (!assign_string(&user_id, item_opt->arg,
- "--user-id")) {
+ if (!assign_string(&owner_uid, item_opt->arg,
+ "--owner-uid")) {
goto error;
}
break;
goto error;
}
- if (user_id) {
+ if (owner_uid) {
char *end;
errno = 0;
- uid = strtol(user_id, &end, 10);
- if (end == user_id || *end != '\0' || errno != 0) {
- ERR("Failed to parse `%s` as an integer.", user_id);
+ uid = strtol(owner_uid, &end, 10);
+ if (end == owner_uid || *end != '\0' || errno != 0) {
+ ERR("Failed to parse `%s` as an integer.", owner_uid);
}
} else {
uid = geteuid();
trigger = lttng_triggers_get_at_index(triggers, i);
trigger_status = lttng_trigger_get_name(trigger, &trigger_name);
- assert(trigger_status == LTTNG_TRIGGER_STATUS_OK);
+ switch (trigger_status) {
+ case LTTNG_TRIGGER_STATUS_OK:
+ break;
+ case LTTNG_TRIGGER_STATUS_UNSET:
+ /* Don't compare against anonymous triggers. */
+ continue;
+ default:
+ abort();
+ }
trigger_status = lttng_trigger_get_owner_uid(
trigger, &trigger_uid);
end:
argpar_parse_ret_fini(&argpar_parse_ret);
lttng_triggers_destroy(triggers);
- free(user_id);
+ free(owner_uid);
return ret;
}