lib: make empty array/map appending/inserting functions return new object
[babeltrace.git] / src / cli / babeltrace2-cfg-cli-args.c
index 13602d45f57ca58418e569912bdbc6b2fb749202..546bdbe28ab33fd14471b32a7275efce063eb2cb 100644 (file)
@@ -42,7 +42,8 @@
 #include "babeltrace2-cfg-cli-args-connect.h"
 #include "babeltrace2-cfg-cli-params-arg.h"
 #include "babeltrace2-plugins.h"
-#include "babeltrace2-cfg-src-auto-disc.h"
+#include "babeltrace2-query.h"
+#include "autodisc/autodisc.h"
 #include "common/version.h"
 
 static const int cli_default_log_level = BT_LOG_WARNING;
@@ -797,7 +798,7 @@ int insert_flat_params_from_array(GString *params_arg,
                goto end;
        }
 
-       for (i = 0; i < bt_value_array_get_size(names_array); i++) {
+       for (i = 0; i < bt_value_array_get_length(names_array); i++) {
                const bt_value *str_obj =
                        bt_value_array_borrow_element_by_index_const(names_array,
                                                                     i);
@@ -1413,9 +1414,9 @@ struct bt_config *bt_config_help_from_args(int argc, const char *argv[],
 {
        int ret, i;
        struct bt_config *cfg = NULL;
-       const char *leftover = NULL;
+       const char *non_opt = NULL;
        char *plugin_name = NULL, *comp_cls_name = NULL;
-       struct bt_argpar_parse_ret argpar_parse_ret;
+       struct bt_argpar_parse_ret argpar_parse_ret = { 0 };
 
        *retcode = 0;
        cfg = bt_config_help_create(initial_plugin_paths, default_log_level);
@@ -1478,18 +1479,18 @@ struct bt_config *bt_config_help_from_args(int argc, const char *argv[],
                        struct bt_argpar_item_non_opt *argpar_item_non_opt
                                = (struct bt_argpar_item_non_opt *) argpar_item;
 
-                       if (leftover) {
+                       if (non_opt) {
                                BT_CLI_LOGE_APPEND_CAUSE("Extraneous command-line argument specified to `help` command: `%s`.",
                                        argpar_item_non_opt->arg);
                                goto error;
                        }
 
-                       leftover = argpar_item_non_opt->arg;
+                       non_opt = argpar_item_non_opt->arg;
                }
        }
 
-       if (leftover) {
-               plugin_comp_cls_names(leftover, NULL,
+       if (non_opt) {
+               plugin_comp_cls_names(non_opt, NULL,
                        &plugin_name, &comp_cls_name,
                        &cfg->cmd_data.help.cfg_component->type);
                if (plugin_name && comp_cls_name) {
@@ -1504,7 +1505,7 @@ struct bt_config *bt_config_help_from_args(int argc, const char *argv[],
                        /* Fall back to plugin help */
                        g_string_assign(
                                cfg->cmd_data.help.cfg_component->plugin_name,
-                               leftover);
+                               non_opt);
                }
        } else {
                print_help_usage(stdout);
@@ -1584,7 +1585,7 @@ struct bt_config *bt_config_query_from_args(int argc, const char *argv[],
        const char *query_object = NULL;
        bt_value *params;
        GString *error_str = NULL;
-       struct bt_argpar_parse_ret argpar_parse_ret;
+       struct bt_argpar_parse_ret argpar_parse_ret = { 0 };
 
        params = bt_value_null;
        bt_value_get_ref(bt_value_null);
@@ -1667,8 +1668,9 @@ struct bt_config *bt_config_query_from_args(int argc, const char *argv[],
                                = (struct bt_argpar_item_non_opt *) argpar_item;
 
                        /*
-                        * We need exactly two leftover arguments which are the
-                        * mandatory component class specification and query object.
+                        * We need exactly two non-option arguments
+                        * which are the mandatory component class
+                        * specification and query object.
                         */
                        if (!component_class_spec) {
                                component_class_spec = argpar_item_non_opt->arg;
@@ -2044,7 +2046,7 @@ struct bt_config *bt_config_run_from_args(int argc, const char *argv[],
                struct bt_argpar_item_opt *argpar_item_opt;
                const char *arg;
 
-               /* This command does not accept leftover arguments. */
+               /* This command does not accept non-option arguments.*/
                if (argpar_item->type == BT_ARGPAR_ITEM_TYPE_NON_OPT) {
                        struct bt_argpar_item_non_opt *argpar_nonopt_item =
                                (struct bt_argpar_item_non_opt *) argpar_item;
@@ -2269,7 +2271,7 @@ struct bt_config *bt_config_run_from_args_array(const bt_value *run_args,
        struct bt_config *cfg = NULL;
        const char **argv;
        int64_t i, len;
-       const size_t argc = bt_value_array_get_size(run_args);
+       const size_t argc = bt_value_array_get_length(run_args);
 
        argv = calloc(argc, sizeof(*argv));
        if (!argv) {
@@ -2277,7 +2279,7 @@ struct bt_config *bt_config_run_from_args_array(const bt_value *run_args,
                goto end;
        }
 
-       len = bt_value_array_get_size(run_args);
+       len = bt_value_array_get_length(run_args);
        if (len < 0) {
                BT_CLI_LOGE_APPEND_CAUSE("Invalid executable arguments.");
                goto end;
@@ -2604,7 +2606,7 @@ int append_run_args_for_implicit_component(
                }
        }
 
-       for (i = 0; i < bt_value_array_get_size(impl_args->extra_params);
+       for (i = 0; i < bt_value_array_get_length(impl_args->extra_params);
                        i++) {
                const bt_value *elem;
                const char *arg;
@@ -2812,7 +2814,7 @@ int bt_value_to_cli_param_value_append(const bt_value *value, GString *buf)
        }
        case BT_VALUE_TYPE_ARRAY: {
                g_string_append_c(buf, '[');
-               uint64_t sz = bt_value_array_get_size(value);
+               uint64_t sz = bt_value_array_get_length(value);
                for (uint64_t i = 0; i < sz; i++) {
                        const bt_value *item;
                        int ret;
@@ -3175,13 +3177,27 @@ end:
 }
 
 /*
- * Create `struct implicit_component_args` structures for each of the source
- * components we identified.  Add them to `component_args`.
+ * Create `struct implicit_component_args` structures for each of the
+ * source components we identified.  Add them to `component_args`.
+ *
+ * `non_opt_params` is an array where each element is an array of
+ * strings containing all the arguments to `--params` that apply to the
+ * non-option argument at the same index.  For example, if, for a
+ * non-option argument, the following `--params` options applied:
+ *
+ *     --params=a=2 --params=b=3,c=4
+ *
+ * its entry in `non_opt_params` would contain
+ *
+ *     ["a=2", "b=3,c=4"]
  */
 
 static
-void create_implicit_component_args_from_auto_discovered_sources(
-               const struct auto_source_discovery *auto_disc, GPtrArray *component_args)
+int create_implicit_component_args_from_auto_discovered_sources(
+               const struct auto_source_discovery *auto_disc,
+               const bt_value *non_opt_params,
+               const bt_value *non_opt_loglevels,
+               GPtrArray *component_args)
 {
        gchar *cc_name = NULL;
        struct implicit_component_args *comp = NULL;
@@ -3193,34 +3209,105 @@ void create_implicit_component_args_from_auto_discovered_sources(
        for (i = 0; i < len; i++) {
                struct auto_source_discovery_result *res =
                        g_ptr_array_index(auto_disc->results, i);
+               uint64_t orig_indices_i, orig_indices_count;
 
                g_free(cc_name);
                cc_name = g_strdup_printf("source.%s.%s", res->plugin_name, res->source_cc_name);
                if (!cc_name) {
                        BT_CLI_LOGE_APPEND_CAUSE_OOM();
-                       goto end;
+                       goto error;
                }
 
                comp = create_implicit_component_args(cc_name);
                if (!comp) {
-                       goto end;
+                       goto error;
+               }
+
+               /*
+                * Append parameters and log levels of all the
+                * non-option arguments that contributed to this
+                * component instance coming into existence.
+                */
+               orig_indices_count = bt_value_array_get_length(res->original_input_indices);
+               for (orig_indices_i = 0; orig_indices_i < orig_indices_count; orig_indices_i++) {
+                       const bt_value *orig_idx_value =
+                               bt_value_array_borrow_element_by_index(
+                                       res->original_input_indices, orig_indices_i);
+                       uint64_t orig_idx = bt_value_integer_unsigned_get(orig_idx_value);
+                       const bt_value *params_array =
+                               bt_value_array_borrow_element_by_index_const(
+                                       non_opt_params, orig_idx);
+                       uint64_t params_i, params_count;
+                       const bt_value *loglevel_value;
+
+                       params_count = bt_value_array_get_length(params_array);
+                       for (params_i = 0; params_i < params_count; params_i++) {
+                               const bt_value *params_value =
+                                       bt_value_array_borrow_element_by_index_const(
+                                               params_array, params_i);
+                               const char *params = bt_value_string_get(params_value);
+                               bt_value_array_append_element_status append_status;
+
+                               append_status = bt_value_array_append_string_element(
+                                       comp->extra_params, "--params");
+                               if (append_status != BT_VALUE_ARRAY_APPEND_ELEMENT_STATUS_OK) {
+                                       BT_CLI_LOGE_APPEND_CAUSE("Failed to append array element.");
+                                       goto error;
+                               }
+
+                               append_status = bt_value_array_append_string_element(
+                                       comp->extra_params, params);
+                               if (append_status != BT_VALUE_ARRAY_APPEND_ELEMENT_STATUS_OK) {
+                                       BT_CLI_LOGE_APPEND_CAUSE("Failed to append array element.");
+                                       goto error;
+                               }
+                       }
+
+                       loglevel_value = bt_value_array_borrow_element_by_index_const(
+                               non_opt_loglevels, orig_idx);
+                       if (bt_value_get_type(loglevel_value) == BT_VALUE_TYPE_STRING) {
+                               const char *loglevel = bt_value_string_get(loglevel_value);
+                               bt_value_array_append_element_status append_status;
+
+                               append_status = bt_value_array_append_string_element(
+                                       comp->extra_params, "--log-level");
+                               if (append_status != BT_VALUE_ARRAY_APPEND_ELEMENT_STATUS_OK) {
+                                       BT_CLI_LOGE_APPEND_CAUSE("Failed to append array element.");
+                                       goto error;
+                               }
+
+                               append_status = bt_value_array_append_string_element(
+                                       comp->extra_params, loglevel);
+                               if (append_status != BT_VALUE_ARRAY_APPEND_ELEMENT_STATUS_OK) {
+                                       BT_CLI_LOGE_APPEND_CAUSE("Failed to append array element.");
+                                       goto error;
+                               }
+                       }
                }
 
                status = append_parameter_to_args(comp->extra_params, "inputs", res->inputs);
                if (status != 0) {
-                       goto end;
+                       goto error;
                }
 
                g_ptr_array_add(component_args, comp);
                comp = NULL;
        }
 
+       status = 0;
+       goto end;
+
+error:
+       status = -1;
+
 end:
        g_free(cc_name);
 
        if (comp) {
                destroy_implicit_component_args(comp);
        }
+
+       return status;
 }
 
 /*
@@ -3234,6 +3321,9 @@ enum convert_current_item_type {
 
        /* Current item is a component. */
        CONVERT_CURRENT_ITEM_TYPE_COMPONENT,
+
+       /* Current item is a non-option argument. */
+       CONVERT_CURRENT_ITEM_TYPE_NON_OPT,
 };
 
 /*
@@ -3265,7 +3355,9 @@ struct bt_config *bt_config_convert_from_args(int argc, const char *argv[],
        GList *source_names = NULL;
        GList *filter_names = NULL;
        GList *sink_names = NULL;
-       bt_value *leftovers = NULL;
+       bt_value *non_opts = NULL;
+       bt_value *non_opt_params = NULL;
+       bt_value *non_opt_loglevels = NULL;
        struct implicit_component_args implicit_ctf_output_args = { 0 };
        struct implicit_component_args implicit_lttng_live_args = { 0 };
        struct implicit_component_args implicit_dummy_args = { 0 };
@@ -3368,8 +3460,20 @@ struct bt_config *bt_config_convert_from_args(int argc, const char *argv[],
                goto error;
        }
 
-       leftovers = bt_value_array_create();
-       if (!leftovers) {
+       non_opts = bt_value_array_create();
+       if (!non_opts) {
+               BT_CLI_LOGE_APPEND_CAUSE_OOM();
+               goto error;
+       }
+
+       non_opt_params = bt_value_array_create();
+       if (!non_opt_params) {
+               BT_CLI_LOGE_APPEND_CAUSE_OOM();
+               goto error;
+       }
+
+       non_opt_loglevels = bt_value_array_create();
+       if (!non_opt_loglevels) {
                BT_CLI_LOGE_APPEND_CAUSE_OOM();
                goto error;
        }
@@ -3424,226 +3528,286 @@ struct bt_config *bt_config_convert_from_args(int argc, const char *argv[],
                char *comp_cls_name = NULL;
                const char *arg;
 
-               if (argpar_item->type != BT_ARGPAR_ITEM_TYPE_OPT) {
-                       continue;
-               }
+               if (argpar_item->type == BT_ARGPAR_ITEM_TYPE_OPT) {
+                       argpar_item_opt = (struct bt_argpar_item_opt *) argpar_item;
+                       arg = argpar_item_opt->arg;
 
-               argpar_item_opt = (struct bt_argpar_item_opt *) argpar_item;
-               arg = argpar_item_opt->arg;
+                       switch (argpar_item_opt->descr->id) {
+                       case OPT_COMPONENT:
+                       {
+                               bt_component_class_type type;
 
-               switch (argpar_item_opt->descr->id) {
-               case OPT_COMPONENT:
-               {
-                       bt_component_class_type type;
+                               current_item_type = CONVERT_CURRENT_ITEM_TYPE_COMPONENT;
 
-                       current_item_type = CONVERT_CURRENT_ITEM_TYPE_COMPONENT;
+                               /* Parse the argument */
+                               plugin_comp_cls_names(arg, &name, &plugin_name,
+                                       &comp_cls_name, &type);
+                               if (!plugin_name || !comp_cls_name) {
+                                       BT_CLI_LOGE_APPEND_CAUSE(
+                                               "Invalid format for --component option's argument:\n    %s",
+                                               arg);
+                                       goto error;
+                               }
 
-                       /* Parse the argument */
-                       plugin_comp_cls_names(arg, &name, &plugin_name,
-                               &comp_cls_name, &type);
-                       if (!plugin_name || !comp_cls_name) {
-                               BT_CLI_LOGE_APPEND_CAUSE(
-                                       "Invalid format for --component option's argument:\n    %s",
-                                       arg);
-                               goto error;
-                       }
+                               if (name) {
+                                       /*
+                                        * Name was given by the user, verify it isn't
+                                        * taken.
+                                        */
+                                       if (bt_value_map_has_entry(all_names, name)) {
+                                               BT_CLI_LOGE_APPEND_CAUSE(
+                                                       "Duplicate component instance name:\n    %s",
+                                                       name);
+                                               goto error;
+                                       }
 
-                       if (name) {
-                               /*
-                                * Name was given by the user, verify it isn't
-                                * taken.
-                                */
-                               if (bt_value_map_has_entry(all_names, name)) {
-                                       BT_CLI_LOGE_APPEND_CAUSE(
-                                               "Duplicate component instance name:\n    %s",
-                                               name);
+                                       name_gstr = g_string_new(name);
+                                       if (!name_gstr) {
+                                               BT_CLI_LOGE_APPEND_CAUSE_OOM();
+                                               goto error;
+                                       }
+
+                                       g_string_assign(component_arg_for_run, arg);
+                               } else {
+                                       /* Name not given by user, generate one. */
+                                       name_gstr = get_component_auto_name(arg, all_names);
+                                       if (!name_gstr) {
+                                               goto error;
+                                       }
+
+                                       g_string_printf(component_arg_for_run, "%s:%s",
+                                               name_gstr->str, arg);
+                               }
+
+                               if (bt_value_array_append_string_element(run_args,
+                                               "--component")) {
+                                       BT_CLI_LOGE_APPEND_CAUSE_OOM();
                                        goto error;
                                }
 
-                               name_gstr = g_string_new(name);
-                               if (!name_gstr) {
+                               if (bt_value_array_append_string_element(run_args,
+                                               component_arg_for_run->str)) {
                                        BT_CLI_LOGE_APPEND_CAUSE_OOM();
                                        goto error;
                                }
 
-                               g_string_assign(component_arg_for_run, arg);
-                       } else {
-                               /* Name not given by user, generate one. */
-                               name_gstr = get_component_auto_name(arg, all_names);
-                               if (!name_gstr) {
+                               /*
+                                * Remember this name globally, for the uniqueness of
+                                * all component names.
+                                */
+                               if (bt_value_map_insert_entry(all_names,
+                                               name_gstr->str, bt_value_null)) {
+                                       BT_CLI_LOGE_APPEND_CAUSE_OOM();
                                        goto error;
                                }
 
-                               g_string_printf(component_arg_for_run, "%s:%s",
-                                       name_gstr->str, arg);
+                               /*
+                                * Remember this name specifically for the type of the
+                                * component. This is to create connection arguments.
+                                *
+                                * The list takes ownership of `name_gstr`.
+                                */
+                               switch (type) {
+                               case BT_COMPONENT_CLASS_TYPE_SOURCE:
+                                       source_names = g_list_append(source_names, name_gstr);
+                                       break;
+                               case BT_COMPONENT_CLASS_TYPE_FILTER:
+                                       filter_names = g_list_append(filter_names, name_gstr);
+                                       break;
+                               case BT_COMPONENT_CLASS_TYPE_SINK:
+                                       sink_names = g_list_append(sink_names, name_gstr);
+                                       break;
+                               default:
+                                       abort();
+                               }
+                               name_gstr = NULL;
+
+                               free(name);
+                               free(plugin_name);
+                               free(comp_cls_name);
+                               name = NULL;
+                               plugin_name = NULL;
+                               comp_cls_name = NULL;
+                               break;
                        }
+                       case OPT_PARAMS:
+                               if (current_item_type == CONVERT_CURRENT_ITEM_TYPE_COMPONENT) {
+                                       /*
+                                        * The current item is a component (--component option),
+                                        * pass it directly to the run args.
+                                        */
+                                       if (bt_value_array_append_string_element(run_args,
+                                                       "--params")) {
+                                               BT_CLI_LOGE_APPEND_CAUSE_OOM();
+                                               goto error;
+                                       }
 
-                       if (bt_value_array_append_string_element(run_args,
-                                       "--component")) {
-                               BT_CLI_LOGE_APPEND_CAUSE_OOM();
-                               goto error;
-                       }
+                                       if (bt_value_array_append_string_element(run_args, arg)) {
+                                               BT_CLI_LOGE_APPEND_CAUSE_OOM();
+                                               goto error;
+                                       }
+                               } else if (current_item_type == CONVERT_CURRENT_ITEM_TYPE_NON_OPT) {
+                                       /*
+                                        * The current item is a
+                                        * non-option argument, record
+                                        * it in `non_opt_params`.
+                                        */
+                                       bt_value *array;
+                                       uint64_t idx = bt_value_array_get_length(non_opt_params) - 1;
 
-                       if (bt_value_array_append_string_element(run_args,
-                                       component_arg_for_run->str)) {
-                               BT_CLI_LOGE_APPEND_CAUSE_OOM();
-                               goto error;
-                       }
+                                       array = bt_value_array_borrow_element_by_index(non_opt_params, idx);
+                                       bt_value_array_append_string_element(array, arg);
+                               } else {
+                                       BT_CLI_LOGE_APPEND_CAUSE(
+                                               "No current component (--component option) or non-option argument of which to set parameters:\n    %s",
+                                               arg);
+                                       goto error;
+                               }
+                               break;
+                       case OPT_LOG_LEVEL:
+                               if (current_item_type == CONVERT_CURRENT_ITEM_TYPE_COMPONENT) {
+                                       if (bt_value_array_append_string_element(run_args, "--log-level")) {
+                                               BT_CLI_LOGE_APPEND_CAUSE_OOM();
+                                               goto error;
+                                       }
 
-                       /*
-                        * Remember this name globally, for the uniqueness of
-                        * all component names.
-                        */
-                       if (bt_value_map_insert_entry(all_names,
-                                       name_gstr->str, bt_value_null)) {
-                               BT_CLI_LOGE_APPEND_CAUSE_OOM();
-                               goto error;
-                       }
+                                       if (bt_value_array_append_string_element(run_args, arg)) {
+                                               BT_CLI_LOGE_APPEND_CAUSE_OOM();
+                                               goto error;
+                                       }
+                               } else if (current_item_type == CONVERT_CURRENT_ITEM_TYPE_NON_OPT) {
+                                       uint64_t idx = bt_value_array_get_length(non_opt_loglevels) - 1;
+                                       bt_value *log_level_str_value;
+
+                                       log_level_str_value = bt_value_string_create_init(arg);
+                                       if (!log_level_str_value) {
+                                               BT_CLI_LOGE_APPEND_CAUSE_OOM();
+                                               goto error;
+                                       }
+
+                                       if (bt_value_array_set_element_by_index(non_opt_loglevels, idx,
+                                                       log_level_str_value)) {
+                                               bt_value_put_ref(log_level_str_value);
+                                               BT_CLI_LOGE_APPEND_CAUSE_OOM();
+                                               goto error;
+                                       }
+                               } else {
+                                       BT_CLI_LOGE_APPEND_CAUSE(
+                                               "No current component (--component option) or non-option argument to assign a log level to:\n    %s",
+                                               arg);
+                                       goto error;
+                               }
 
-                       /*
-                        * Remember this name specifically for the type of the
-                        * component. This is to create connection arguments.
-                        *
-                        * The list takes ownership of `name_gstr`.
-                        */
-                       switch (type) {
-                       case BT_COMPONENT_CLASS_TYPE_SOURCE:
-                               source_names = g_list_append(source_names, name_gstr);
-                               break;
-                       case BT_COMPONENT_CLASS_TYPE_FILTER:
-                               filter_names = g_list_append(filter_names, name_gstr);
                                break;
-                       case BT_COMPONENT_CLASS_TYPE_SINK:
-                               sink_names = g_list_append(sink_names, name_gstr);
+                       case OPT_OMIT_HOME_PLUGIN_PATH:
+                               force_omit_home_plugin_path = true;
+
+                               if (bt_value_array_append_string_element(run_args,
+                                               "--omit-home-plugin-path")) {
+                                       BT_CLI_LOGE_APPEND_CAUSE_OOM();
+                                       goto error;
+                               }
                                break;
-                       default:
-                               abort();
-                       }
-                       name_gstr = NULL;
-
-                       free(name);
-                       free(plugin_name);
-                       free(comp_cls_name);
-                       name = NULL;
-                       plugin_name = NULL;
-                       comp_cls_name = NULL;
-                       break;
-               }
-               case OPT_PARAMS:
-                       if (current_item_type != CONVERT_CURRENT_ITEM_TYPE_COMPONENT) {
-                               BT_CLI_LOGE_APPEND_CAUSE("No current component of which to set parameters:\n    %s",
-                                       arg);
-                               goto error;
-                       }
+                       case OPT_RETRY_DURATION:
+                               if (bt_value_array_append_string_element(run_args,
+                                               "--retry-duration")) {
+                                       BT_CLI_LOGE_APPEND_CAUSE_OOM();
+                                       goto error;
+                               }
 
-                       if (bt_value_array_append_string_element(run_args,
-                                       "--params")) {
-                               BT_CLI_LOGE_APPEND_CAUSE_OOM();
-                               goto error;
-                       }
+                               if (bt_value_array_append_string_element(run_args, arg)) {
+                                       BT_CLI_LOGE_APPEND_CAUSE_OOM();
+                                       goto error;
+                               }
+                               break;
+                       case OPT_OMIT_SYSTEM_PLUGIN_PATH:
+                               force_omit_system_plugin_path = true;
 
-                       if (bt_value_array_append_string_element(run_args, arg)) {
-                               BT_CLI_LOGE_APPEND_CAUSE_OOM();
-                               goto error;
-                       }
-                       break;
-               case OPT_LOG_LEVEL:
-                       if (current_item_type != CONVERT_CURRENT_ITEM_TYPE_COMPONENT) {
-                               BT_CLI_LOGE_APPEND_CAUSE("No current component to assign a log level to:\n    %s",
-                                       arg);
-                               goto error;
-                       }
+                               if (bt_value_array_append_string_element(run_args,
+                                               "--omit-system-plugin-path")) {
+                                       BT_CLI_LOGE_APPEND_CAUSE_OOM();
+                                       goto error;
+                               }
+                               break;
+                       case OPT_PLUGIN_PATH:
+                               if (bt_config_append_plugin_paths_check_setuid_setgid(
+                                               plugin_paths, arg)) {
+                                       goto error;
+                               }
 
-                       if (bt_value_array_append_string_element(run_args, "--log-level")) {
-                               BT_CLI_LOGE_APPEND_CAUSE_OOM();
-                               goto error;
-                       }
+                               if (bt_value_array_append_string_element(run_args,
+                                               "--plugin-path")) {
+                                       BT_CLI_LOGE_APPEND_CAUSE_OOM();
+                                       goto error;
+                               }
 
-                       if (bt_value_array_append_string_element(run_args, arg)) {
-                               BT_CLI_LOGE_APPEND_CAUSE_OOM();
+                               if (bt_value_array_append_string_element(run_args, arg)) {
+                                       BT_CLI_LOGE_APPEND_CAUSE_OOM();
+                                       goto error;
+                               }
+                               break;
+                       case OPT_BEGIN:
+                       case OPT_CLOCK_CYCLES:
+                       case OPT_CLOCK_DATE:
+                       case OPT_CLOCK_FORCE_CORRELATE:
+                       case OPT_CLOCK_GMT:
+                       case OPT_CLOCK_OFFSET:
+                       case OPT_CLOCK_OFFSET_NS:
+                       case OPT_CLOCK_SECONDS:
+                       case OPT_COLOR:
+                       case OPT_DEBUG:
+                       case OPT_DEBUG_INFO:
+                       case OPT_DEBUG_INFO_DIR:
+                       case OPT_DEBUG_INFO_FULL_PATH:
+                       case OPT_DEBUG_INFO_TARGET_PREFIX:
+                       case OPT_END:
+                       case OPT_FIELDS:
+                       case OPT_INPUT_FORMAT:
+                       case OPT_NAMES:
+                       case OPT_NO_DELTA:
+                       case OPT_OUTPUT_FORMAT:
+                       case OPT_OUTPUT:
+                       case OPT_RUN_ARGS:
+                       case OPT_RUN_ARGS_0:
+                       case OPT_STREAM_INTERSECTION:
+                       case OPT_TIMERANGE:
+                       case OPT_VERBOSE:
+                               /* Ignore in this pass */
+                               break;
+                       default:
+                               BT_CLI_LOGE_APPEND_CAUSE("Unknown command-line option specified (option code %d).",
+                                       argpar_item_opt->descr->id);
                                goto error;
                        }
+               } else if (argpar_item->type == BT_ARGPAR_ITEM_TYPE_NON_OPT) {
+                       struct bt_argpar_item_non_opt *argpar_item_non_opt;
+                       bt_value_array_append_element_status append_status;
 
-                       break;
-               case OPT_OMIT_HOME_PLUGIN_PATH:
-                       force_omit_home_plugin_path = true;
+                       current_item_type = CONVERT_CURRENT_ITEM_TYPE_NON_OPT;
 
-                       if (bt_value_array_append_string_element(run_args,
-                                       "--omit-home-plugin-path")) {
-                               BT_CLI_LOGE_APPEND_CAUSE_OOM();
-                               goto error;
-                       }
-                       break;
-               case OPT_RETRY_DURATION:
-                       if (bt_value_array_append_string_element(run_args,
-                                       "--retry-duration")) {
-                               BT_CLI_LOGE_APPEND_CAUSE_OOM();
-                               goto error;
-                       }
+                       argpar_item_non_opt = (struct bt_argpar_item_non_opt *) argpar_item;
 
-                       if (bt_value_array_append_string_element(run_args, arg)) {
-                               BT_CLI_LOGE_APPEND_CAUSE_OOM();
-                               goto error;
-                       }
-                       break;
-               case OPT_OMIT_SYSTEM_PLUGIN_PATH:
-                       force_omit_system_plugin_path = true;
-
-                       if (bt_value_array_append_string_element(run_args,
-                                       "--omit-system-plugin-path")) {
+                       append_status = bt_value_array_append_string_element(non_opts,
+                               argpar_item_non_opt->arg);
+                       if (append_status != BT_VALUE_ARRAY_APPEND_ELEMENT_STATUS_OK) {
                                BT_CLI_LOGE_APPEND_CAUSE_OOM();
                                goto error;
                        }
-                       break;
-               case OPT_PLUGIN_PATH:
-                       if (bt_config_append_plugin_paths_check_setuid_setgid(
-                                       plugin_paths, arg)) {
-                               goto error;
-                       }
 
-                       if (bt_value_array_append_string_element(run_args,
-                                       "--plugin-path")) {
+                       append_status = bt_value_array_append_empty_array_element(
+                               non_opt_params, NULL);
+                       if (append_status != BT_VALUE_ARRAY_APPEND_ELEMENT_STATUS_OK) {
                                BT_CLI_LOGE_APPEND_CAUSE_OOM();
                                goto error;
                        }
 
-                       if (bt_value_array_append_string_element(run_args, arg)) {
+                       append_status = bt_value_array_append_element(non_opt_loglevels, bt_value_null);
+                       if (append_status != BT_VALUE_ARRAY_APPEND_ELEMENT_STATUS_OK) {
                                BT_CLI_LOGE_APPEND_CAUSE_OOM();
                                goto error;
                        }
-                       break;
-               case OPT_BEGIN:
-               case OPT_CLOCK_CYCLES:
-               case OPT_CLOCK_DATE:
-               case OPT_CLOCK_FORCE_CORRELATE:
-               case OPT_CLOCK_GMT:
-               case OPT_CLOCK_OFFSET:
-               case OPT_CLOCK_OFFSET_NS:
-               case OPT_CLOCK_SECONDS:
-               case OPT_COLOR:
-               case OPT_DEBUG:
-               case OPT_DEBUG_INFO:
-               case OPT_DEBUG_INFO_DIR:
-               case OPT_DEBUG_INFO_FULL_PATH:
-               case OPT_DEBUG_INFO_TARGET_PREFIX:
-               case OPT_END:
-               case OPT_FIELDS:
-               case OPT_INPUT_FORMAT:
-               case OPT_NAMES:
-               case OPT_NO_DELTA:
-               case OPT_OUTPUT_FORMAT:
-               case OPT_OUTPUT:
-               case OPT_RUN_ARGS:
-               case OPT_RUN_ARGS_0:
-               case OPT_STREAM_INTERSECTION:
-               case OPT_TIMERANGE:
-               case OPT_VERBOSE:
-                       /* Ignore in this pass */
-                       break;
-               default:
-                       BT_CLI_LOGE_APPEND_CAUSE("Unknown command-line option specified (option code %d).",
-                               argpar_item_opt->descr->id);
-                       goto error;
+               } else {
+                       abort();
                }
        }
 
@@ -3964,35 +4128,16 @@ struct bt_config *bt_config_convert_from_args(int argc, const char *argv[],
                goto error;
        }
 
-       /* Consume and keep leftover arguments */
-       for (i = 0; i < argpar_parse_ret.items->len; i++) {
-               struct bt_argpar_item *argpar_item =
-                       g_ptr_array_index(argpar_parse_ret.items, i);
-               struct bt_argpar_item_non_opt *argpar_item_non_opt;
-
-               if (argpar_item->type != BT_ARGPAR_ITEM_TYPE_NON_OPT) {
-                       continue;
-               }
-
-               argpar_item_non_opt = (struct bt_argpar_item_non_opt *) argpar_item;
-
-               if (bt_value_array_append_string_element(leftovers, argpar_item_non_opt->arg) !=
-                               BT_VALUE_ARRAY_APPEND_ELEMENT_STATUS_OK) {
-                       BT_CLI_LOGE_APPEND_CAUSE_OOM();
-                       goto error;
-               }
-       }
-
        /* Print CTF metadata or print LTTng live sessions */
        if (print_ctf_metadata) {
-               const bt_value *bt_val_leftover;
+               const bt_value *bt_val_non_opt;
 
-               if (bt_value_array_is_empty(leftovers)) {
+               if (bt_value_array_is_empty(non_opts)) {
                        BT_CLI_LOGE_APPEND_CAUSE("--output-format=ctf-metadata specified without a path.");
                        goto error;
                }
 
-               if (bt_value_array_get_size(leftovers) > 1) {
+               if (bt_value_array_get_length(non_opts) > 1) {
                        BT_CLI_LOGE_APPEND_CAUSE("Too many paths specified for --output-format=ctf-metadata.");
                        goto error;
                }
@@ -4002,9 +4147,9 @@ struct bt_config *bt_config_convert_from_args(int argc, const char *argv[],
                        goto error;
                }
 
-               bt_val_leftover = bt_value_array_borrow_element_by_index_const(leftovers, 0);
+               bt_val_non_opt = bt_value_array_borrow_element_by_index_const(non_opts, 0);
                g_string_assign(cfg->cmd_data.print_ctf_metadata.path,
-                               bt_value_string_get(bt_val_leftover));
+                               bt_value_string_get(bt_val_non_opt));
 
                if (output) {
                        g_string_assign(
@@ -4069,19 +4214,19 @@ struct bt_config *bt_config_convert_from_args(int argc, const char *argv[],
                }
        }
 
-       /* Decide where the leftover argument(s) go */
-       if (bt_value_array_get_size(leftovers) > 0) {
+       /* Decide where the non-option argument(s) go */
+       if (bt_value_array_get_length(non_opts) > 0) {
                if (implicit_lttng_live_args.exists) {
-                       const bt_value *bt_val_leftover;
+                       const bt_value *bt_val_non_opt;
 
-                       if (bt_value_array_get_size(leftovers) > 1) {
+                       if (bt_value_array_get_length(non_opts) > 1) {
                                BT_CLI_LOGE_APPEND_CAUSE("Too many URLs specified for --input-format=lttng-live.");
                                goto error;
                        }
 
-                       bt_val_leftover = bt_value_array_borrow_element_by_index_const(leftovers, 0);
+                       bt_val_non_opt = bt_value_array_borrow_element_by_index_const(non_opts, 0);
                        lttng_live_url_parts =
-                               bt_common_parse_lttng_live_url(bt_value_string_get(bt_val_leftover),
+                               bt_common_parse_lttng_live_url(bt_value_string_get(bt_val_non_opt),
                                        error_buf, sizeof(error_buf));
                        if (!lttng_live_url_parts.proto) {
                                BT_CLI_LOGE_APPEND_CAUSE("Invalid LTTng live URL format: %s.",
@@ -4098,7 +4243,7 @@ struct bt_config *bt_config_convert_from_args(int argc, const char *argv[],
                                }
 
                                g_string_assign(cfg->cmd_data.print_lttng_live_sessions.url,
-                                       bt_value_string_get(bt_val_leftover));
+                                       bt_value_string_get(bt_val_non_opt));
 
                                if (output) {
                                        g_string_assign(
@@ -4111,7 +4256,7 @@ struct bt_config *bt_config_convert_from_args(int argc, const char *argv[],
 
                        ret = append_implicit_component_extra_param(
                                &implicit_lttng_live_args, "url",
-                               bt_value_string_get(bt_val_leftover));
+                               bt_value_string_get(bt_val_non_opt));
                        if (ret) {
                                goto error;
                        }
@@ -4124,9 +4269,25 @@ struct bt_config *bt_config_convert_from_args(int argc, const char *argv[],
                        }
                } else {
                        int status;
+                       size_t plugin_count;
+                       const bt_plugin **plugins;
+                       const bt_plugin *plugin;
+
+                       status = require_loaded_plugins(plugin_paths);
+                       if (status != 0) {
+                               goto error;
+                       }
+
+                       if (auto_source_discovery_restrict_plugin_name) {
+                               plugin_count = 1;
+                               plugin = find_loaded_plugin(auto_source_discovery_restrict_plugin_name);
+                               plugins = &plugin;
+                       } else {
+                               plugin_count = get_loaded_plugins_count();
+                               plugins = borrow_loaded_plugins();
+                       }
 
-                       status = auto_discover_source_components(plugin_paths, leftovers,
-                               auto_source_discovery_restrict_plugin_name,
+                       status = auto_discover_source_components(non_opts, plugins, plugin_count,
                                auto_source_discovery_restrict_component_class_name,
                                *default_log_level >= 0 ? *default_log_level : cli_default_log_level,
                                &auto_disc);
@@ -4135,8 +4296,12 @@ struct bt_config *bt_config_convert_from_args(int argc, const char *argv[],
                                goto error;
                        }
 
-                       create_implicit_component_args_from_auto_discovered_sources(
-                               &auto_disc, discovered_source_args);
+                       status = create_implicit_component_args_from_auto_discovered_sources(
+                               &auto_disc, non_opt_params, non_opt_loglevels,
+                               discovered_source_args);
+                       if (status != 0) {
+                               goto error;
+                       }
                }
        }
 
@@ -4169,10 +4334,11 @@ struct bt_config *bt_config_convert_from_args(int argc, const char *argv[],
        }
 
        /*
-        * If the implicit `source.ctf.lttng-live` component exists, make sure
-        * there's at least one leftover (which is the URL).
+        * If the implicit `source.ctf.lttng-live` component exists,
+        * make sure there's at least one non-option argument (which is
+        * the URL).
         */
-       if (implicit_lttng_live_args.exists && bt_value_array_is_empty(leftovers)) {
+       if (implicit_lttng_live_args.exists && bt_value_array_is_empty(non_opts)) {
                BT_CLI_LOGE_APPEND_CAUSE("Missing URL for implicit `%s` component.",
                        implicit_lttng_live_args.comp_arg->str);
                goto error;
@@ -4355,7 +4521,7 @@ struct bt_config *bt_config_convert_from_args(int argc, const char *argv[],
                        goto error;
                }
 
-               for (i = 0; i < bt_value_array_get_size(run_args); i++) {
+               for (i = 0; i < bt_value_array_get_length(run_args); i++) {
                        const bt_value *arg_value =
                                bt_value_array_borrow_element_by_index(run_args,
                                                                       i);
@@ -4383,7 +4549,7 @@ struct bt_config *bt_config_convert_from_args(int argc, const char *argv[],
                                g_string_free(quoted, TRUE);
                        }
 
-                       if (i < bt_value_array_get_size(run_args) - 1) {
+                       if (i < bt_value_array_get_length(run_args) - 1) {
                                if (print_run_args) {
                                        putchar(' ');
                                } else {
@@ -4446,7 +4612,9 @@ end:
        destroy_glist_of_gstring(source_names);
        destroy_glist_of_gstring(filter_names);
        destroy_glist_of_gstring(sink_names);
-       bt_value_put_ref(leftovers);
+       bt_value_put_ref(non_opt_params);
+       bt_value_put_ref(non_opt_loglevels);
+       bt_value_put_ref(non_opts);
        finalize_implicit_component_args(&implicit_ctf_output_args);
        finalize_implicit_component_args(&implicit_lttng_live_args);
        finalize_implicit_component_args(&implicit_dummy_args);
This page took 0.036419 seconds and 4 git commands to generate.