#include <string.h>
#include <stdarg.h>
-#include "../command.h"
-#include "../loglevel.h"
-#include "../uprobe.h"
+#include "../command.hpp"
+#include "../loglevel.hpp"
+#include "../uprobe.hpp"
#include "common/argpar/argpar.h"
-#include "common/argpar-utils/argpar-utils.h"
-#include "common/dynamic-array.h"
-#include "common/mi-lttng.h"
-#include "common/string-utils/string-utils.h"
-#include "common/utils.h"
-#include <lttng/domain-internal.h>
+#include "common/argpar-utils/argpar-utils.hpp"
+#include "common/dynamic-array.hpp"
+#include "common/mi-lttng.hpp"
+#include "common/string-utils/string-utils.hpp"
+#include "common/utils.hpp"
+#include <lttng/domain-internal.hpp>
/* For lttng_event_rule_type_str(). */
-#include <lttng/event-rule/event-rule-internal.h>
+#include <lttng/event-rule/event-rule-internal.hpp>
#include <lttng/lttng.h>
-#include "common/filter/filter-ast.h"
-#include "common/filter/filter-ir.h"
-#include "common/dynamic-array.h"
+#include "common/filter/filter-ast.hpp"
+#include "common/filter/filter-ir.hpp"
+#include "common/dynamic-array.hpp"
#if (LTTNG_SYMBOL_NAME_LEN == 256)
#define LTTNG_SYMBOL_NAME_LEN_SCANF_IS_A_BROKEN_API "255"
PERROR("Failed to copy kernel probe location symbol name.");
goto error;
}
- offset = strtoul(s_hex, NULL, 0);
+ offset = strtoull(s_hex, NULL, 0);
*location = lttng_kernel_probe_location_symbol_create(
symbol_name, offset);
goto error;
}
- address = strtoul(s_hex, NULL, 0);
+ address = strtoull(s_hex, NULL, 0);
*location = lttng_kernel_probe_location_address_create(address);
if (!*location) {
ERR("Failed to create symbol kernel probe location.");
int consumed_args = -1;
struct lttng_kernel_probe_location *kernel_probe_location = NULL;
struct lttng_userspace_probe_location *userspace_probe_location = NULL;
- struct parse_event_rule_res res = { 0 };
+ struct parse_event_rule_res res = {};
struct lttng_event_expr *event_expr = NULL;
struct filter_parser_ctx *parser_ctx = NULL;
struct lttng_log_level_rule *log_level_rule = NULL;
enum parse_next_item_status status;
status = parse_next_item(argpar_iter, &argpar_item,
- argc_offset, *argv, false, NULL);
- if (status == PARSE_NEXT_ITEM_STATUS_ERROR) {
+ argc_offset, *argv, false, NULL, NULL);
+ if (status == PARSE_NEXT_ITEM_STATUS_ERROR ||
+ status == PARSE_NEXT_ITEM_STATUS_ERROR_MEMORY) {
goto error;
} else if (status == PARSE_NEXT_ITEM_STATUS_END) {
break;
{ "event-rule-matches", handle_condition_event },
};
+static
+void print_valid_condition_names(void)
+{
+ unsigned int i;
+
+ ERR("Valid condition names are:");
+
+ for (i = 0; i < ARRAY_SIZE(condition_descrs); ++i) {
+ ERR(" %s", condition_descrs[i].name);
+ }
+}
+
static
struct lttng_condition *parse_condition(const char *condition_name, int *argc,
const char ***argv, int argc_offset, int orig_arg_index,
if (!descr) {
ERR(WHILE_PARSING_ARG_N_ARG_FMT "Unknown condition name '%s'",
orig_arg_index + 1, orig_arg, condition_name);
+ print_valid_condition_names();
goto error;
}
enum parse_next_item_status status;
status = parse_next_item(argpar_iter, &argpar_item,
- argc_offset, *argv, false,
+ argc_offset, *argv, false, NULL,
"While parsing `notify` action:");
- if (status == PARSE_NEXT_ITEM_STATUS_ERROR) {
+ if (status == PARSE_NEXT_ITEM_STATUS_ERROR ||
+ status == PARSE_NEXT_ITEM_STATUS_ERROR_MEMORY) {
goto error;
} else if (status == PARSE_NEXT_ITEM_STATUS_END) {
break;
enum parse_next_item_status status;
status = parse_next_item(argpar_iter, &argpar_item, argc_offset,
- *argv, false,
+ *argv, false, NULL,
"While parsing `%s` action:", action_name);
- if (status == PARSE_NEXT_ITEM_STATUS_ERROR) {
+ if (status == PARSE_NEXT_ITEM_STATUS_ERROR ||
+ status == PARSE_NEXT_ITEM_STATUS_ERROR_MEMORY) {
goto error;
} else if (status == PARSE_NEXT_ITEM_STATUS_END) {
break;
enum parse_next_item_status status;
status = parse_next_item(argpar_iter, &argpar_item, argc_offset,
- *argv, false, "While parsing `snapshot` action:");
- if (status == PARSE_NEXT_ITEM_STATUS_ERROR) {
+ *argv, false, NULL, "While parsing `snapshot` action:");
+ if (status == PARSE_NEXT_ITEM_STATUS_ERROR ||
+ status == PARSE_NEXT_ITEM_STATUS_ERROR_MEMORY) {
goto error;
} else if (status == PARSE_NEXT_ITEM_STATUS_END) {
break;
{ "snapshot-session", handle_action_snapshot_session },
};
+static
+void print_valid_action_names(void)
+{
+ unsigned int i;
+
+ ERR("Valid action names are:");
+
+ for (i = 0; i < ARRAY_SIZE(condition_descrs); ++i) {
+ ERR(" %s", action_descrs[i].name);
+ }
+}
+
static
struct lttng_action *parse_action(const char *action_name, int *argc,
const char ***argv, int argc_offset, int orig_arg_index,
}
if (!descr) {
- ERR(WHILE_PARSING_ARG_N_ARG_FMT "Unknown action name: %s",
+ ERR(WHILE_PARSING_ARG_N_ARG_FMT "Unknown action name '%s'",
orig_arg_index + 1, orig_arg, action_name);
+ print_valid_action_names();
goto error;
}
struct lttng_dynamic_pointer_array actions;
struct argpar_iter *argpar_iter = NULL;
const struct argpar_item *argpar_item = NULL;
+ const struct argpar_error *argpar_error = NULL;
struct lttng_action *action_list = NULL;
struct lttng_action *action = NULL;
struct lttng_trigger *trigger = NULL;
}
status = parse_next_item(argpar_iter, &argpar_item,
- argc - my_argc, my_argv, true, NULL);
+ argc - my_argc, my_argv, true, &argpar_error, NULL);
if (status == PARSE_NEXT_ITEM_STATUS_ERROR) {
+
+ if (argpar_error_type(argpar_error) ==
+ ARGPAR_ERROR_TYPE_MISSING_OPT_ARG) {
+ int opt_id = argpar_error_opt_descr(argpar_error, NULL)->id;
+
+ if (opt_id == OPT_CONDITION) {
+ print_valid_condition_names();
+ } else if (opt_id == OPT_ACTION) {
+ print_valid_action_names();
+ }
+ }
+
+ goto error;
+ } else if (status == PARSE_NEXT_ITEM_STATUS_ERROR_MEMORY) {
goto error;
} else if (status == PARSE_NEXT_ITEM_STATUS_END) {
break;
}
cleanup:
+ argpar_error_destroy(argpar_error);
argpar_iter_destroy(argpar_iter);
argpar_item_destroy(argpar_item);
lttng_dynamic_pointer_array_reset(&actions);