babeltrace.c: replace printf_verbose() with printf() where appropriate
[babeltrace.git] / converter / babeltrace-cfg.c
index 5811f4447d06ab955853196d903717ea91224442..f045bf94e973b6c4c16ef989b64cb4e516d8eff9 100644 (file)
@@ -30,6 +30,7 @@
 #include <stdbool.h>
 #include <inttypes.h>
 #include <babeltrace/babeltrace.h>
+#include <babeltrace/common-internal.h>
 #include <babeltrace/values.h>
 #include <popt.h>
 #include <glib.h>
 #include <pwd.h>
 #include "babeltrace-cfg.h"
 
-#define SYSTEM_PLUGIN_PATH             INSTALL_LIBDIR "/babeltrace/plugins"
 #define DEFAULT_SOURCE_COMPONENT_NAME  "ctf.fs"
 #define DEFAULT_SINK_COMPONENT_NAME    "text.text"
-#define HOME_ENV_VAR                   "HOME"
-#define HOME_SUBPATH                   "/.babeltrace/plugins"
 
 /*
  * Error printf() macro which prepends "Error: " the first time it's
@@ -766,16 +764,23 @@ void print_usage(FILE *fp)
        fprintf(fp, "                                    of the following source and sink component\n");
        fprintf(fp, "                                    instances (see the exact format of PARAMS\n");
        fprintf(fp, "                                    below)\n");
+       fprintf(fp, "      --begin=BEGIN                 Set beginning time to BEGIN\n");
+       fprintf(fp, "                                    (format: [YYYY-MM-DD [hh:mm:]]ss[.nnnnnnnnn])\n");
        fprintf(fp, "  -d, --debug                       Enable debug mode\n");
+       fprintf(fp, "      --end=END                     Set end time to END\n");
+       fprintf(fp, "                                    (format: [YYYY-MM-DD [hh:mm:]]ss[.nnnnnnnnn])\n");
        fprintf(fp, "  -l, --list                        List available plugins and their components\n");
-       fprintf(fp, "  -P, --path=PATH                   Set the `path` parameter of the latest source\n");
-       fprintf(fp, "                                    or sink component to PATH\n");
+       fprintf(fp, "      --omit-home-plugin-path       Omit home plugins from plugin search path\n");
+       fprintf(fp, "                                    (~/.local/lib/babeltrace/plugins)\n");
+       fprintf(fp, "      --omit-system-plugin-path     Omit system plugins from plugin search path\n");
        fprintf(fp, "  -p, --params=PARAMS               Set the parameters of the latest source or\n");
        fprintf(fp, "                                    sink component instance (in command-line \n");
        fprintf(fp, "                                    order) to PARAMS (see the exact format of\n");
        fprintf(fp, "                                    PARAMS below)\n");
-       fprintf(fp, "      --plugin-path=PATH[:PATH]...  Set paths from which dynamic plugins can be\n");
-       fprintf(fp, "                                    loaded to PATH\n");
+       fprintf(fp, "  -P, --path=PATH                   Set the `path` parameter of the latest source\n");
+       fprintf(fp, "                                    or sink component to PATH\n");
+       fprintf(fp, "      --plugin-path=PATH[:PATH]...  Add PATH to the list of paths from which dynamic\n");
+       fprintf(fp, "                                    plugins can be loaded\n");
        fprintf(fp, "  -r, --reset-base-params           Reset the current base parameters of the\n");
        fprintf(fp, "                                    following source and sink component\n");
        fprintf(fp, "                                    instances to an empty map\n");
@@ -785,11 +790,8 @@ void print_usage(FILE *fp)
        fprintf(fp, "  -i, --source=PLUGIN.COMPCLS       Instantiate a source component from plugin\n");
        fprintf(fp, "                                    PLUGIN and component class COMPCLS (may be\n");
        fprintf(fp, "                                    repeated)\n");
-       fprintf(fp, "      --begin                       Start time: [YYYY-MM-DD [hh:mm:]]ss[.nnnnnnnnn]\n");
-       fprintf(fp, "      --end                         End time: [YYYY-MM-DD [hh:mm:]]ss[.nnnnnnnnn]\n");
-       fprintf(fp, "      --timerange                   Time range: begin,end or [begin,end] (where [] are actual brackets)\n");
-       fprintf(fp, "      --omit-system-plugin-path     Omit system plugins from plugin search path\n");
-       fprintf(fp, "      --omit-home-plugin-path       Omit home plugins from plugin search path\n");
+       fprintf(fp, "      --timerange=TIMERANGE         Set time range to TIMERANGE: BEGIN,END or\n");
+       fprintf(fp, "                                    [BEGIN,END] (where [ and ] are actual brackets)\n");
        fprintf(fp, "  -h  --help                        Show this help\n");
        fprintf(fp, "      --help-legacy                 Show Babeltrace 1.x legacy options\n");
        fprintf(fp, "  -v, --verbose                     Enable verbose output\n");
@@ -964,64 +966,43 @@ end:
        return;
 }
 
-static
-bool is_setuid_setgid(void)
+static void destroy_gstring(void *data)
 {
-       return (geteuid() != getuid() || getegid() != getgid());
+       g_string_free(data, TRUE);
 }
 
 /*
  * Extracts the various paths from the string arg, delimited by ':',
- * and converts them to an array value object.
- *
- * Returned array value object is empty if arg is empty.
- *
- * Return value is owned by the caller.
+ * and appends them to the array value object plugin_paths.
  */
-static
-enum bt_value_status plugin_paths_from_arg(struct bt_value *plugin_paths,
-               const char *arg)
+enum bt_value_status bt_config_append_plugin_paths(
+               struct bt_value *plugin_paths, const char *arg)
 {
-       const char *at = arg;
-       const char *end = arg + strlen(arg);
+       enum bt_value_status status = BT_VALUE_STATUS_OK;
+       GPtrArray *dirs = g_ptr_array_new_with_free_func(destroy_gstring);
+       int ret;
+       size_t i;
 
-       while (at < end) {
-               int ret;
-               GString *path;
-               const char *next_colon;
+       if (!dirs) {
+               status = BT_VALUE_STATUS_ERROR;
+               goto end;
+       }
 
-               next_colon = strchr(at, ':');
-               if (next_colon == at) {
-                       /*
-                        * Empty path: try next character (supported
-                        * to conform to the typical parsing of $PATH).
-                        */
-                       at++;
-                       continue;
-               } else if (!next_colon) {
-                       /* No more colon: use the remaining */
-                       next_colon = arg + strlen(arg);
-               }
+       ret = bt_common_append_plugin_path_dirs(arg, dirs);
+       if (ret) {
+               status = BT_VALUE_STATUS_ERROR;
+               goto end;
+       }
 
-               path = g_string_new(NULL);
-               if (!path) {
-                       print_err_oom();
-                       goto error;
-               }
+       for (i = 0; i < dirs->len; i++) {
+               GString *dir = g_ptr_array_index(dirs, i);
 
-               g_string_append_len(path, at, next_colon - at);
-               at = next_colon + 1;
-               ret = bt_value_array_append_string(plugin_paths, path->str);
-               g_string_free(path, TRUE);
-               if (ret) {
-                       print_err_oom();
-                       goto error;
-               }
+               bt_value_array_append_string(plugin_paths, dir->str);
        }
 
-       return BT_VALUE_STATUS_OK;
-error:
-       return BT_VALUE_STATUS_ERROR;
+end:
+       g_ptr_array_free(dirs, TRUE);
+       return status;
 }
 
 /*
@@ -2254,53 +2235,44 @@ not_found:
        return -1;
 }
 
-static char *bt_secure_getenv(const char *name)
+static int add_env_var_plugin_paths(struct bt_config *cfg)
 {
-       if (is_setuid_setgid()) {
-               printf_err("Disregarding %s environment variable for setuid/setgid binary", name);
-               return NULL;
-       }
-       return getenv(name);
-}
-
-static const char *get_home_dir(void)
-{
-       char *val = NULL;
-       struct passwd *pwd;
+       int ret = 0;
+       const char *envvar;
 
-       val = bt_secure_getenv(HOME_ENV_VAR);
-       if (val) {
+       if (bt_common_is_setuid_setgid()) {
+               printf_debug("Skipping non-system plugin paths for setuid/setgid binary\n");
                goto end;
        }
-       /* Fallback on password file. */
-       pwd = getpwuid(getuid());
-       if (!pwd) {
+
+       envvar = getenv("BABELTRACE_PLUGIN_PATH");
+       if (!envvar) {
                goto end;
        }
-       val = pwd->pw_dir;
+
+       ret = bt_config_append_plugin_paths(cfg->plugin_paths, envvar);
+
 end:
-       return val;
+       return ret;
 }
 
-static int add_internal_plugin_paths(struct bt_config *cfg)
+static int append_home_and_system_plugin_paths(struct bt_config *cfg)
 {
-       if (!cfg->omit_home_plugin_path) {
-               char path[PATH_MAX];
-               const char *home_dir;
+       int ret;
 
-               if (is_setuid_setgid()) {
-                       printf_debug("Skipping non-system plugin paths for setuid/setgid binary.");
+       if (!cfg->omit_home_plugin_path) {
+               if (bt_common_is_setuid_setgid()) {
+                       printf_debug("Skipping non-system plugin paths for setuid/setgid binary\n");
                } else {
-                       home_dir = get_home_dir();
-                       if (home_dir) {
-                               if (strlen(home_dir) + strlen(HOME_SUBPATH) + 1
-                                               >= PATH_MAX) {
-                                       printf_err("Home directory path too long\n");
-                                       goto error;
-                               }
-                               strcpy(path, home_dir);
-                               strcat(path, HOME_SUBPATH);
-                               if (plugin_paths_from_arg(cfg->plugin_paths, path)) {
+                       char *home_plugin_dir =
+                               bt_common_get_home_plugin_path();
+
+                       if (home_plugin_dir) {
+                               ret = bt_config_append_plugin_paths(cfg->plugin_paths,
+                                       home_plugin_dir);
+                               free(home_plugin_dir);
+
+                               if (ret) {
                                        printf_err("Invalid home plugin path\n");
                                        goto error;
                                }
@@ -2309,8 +2281,8 @@ static int add_internal_plugin_paths(struct bt_config *cfg)
        }
 
        if (!cfg->omit_system_plugin_path) {
-               if (plugin_paths_from_arg(cfg->plugin_paths,
-                               SYSTEM_PLUGIN_PATH)) {
+               if (bt_config_append_plugin_paths(cfg->plugin_paths,
+                               bt_common_get_system_plugin_path())) {
                        printf_err("Invalid system plugin path\n");
                        goto error;
                }
@@ -2381,9 +2353,8 @@ error:
 }
 
 /*
- * Returns a Babeltrace configuration, out of command-line arguments,
- * containing everything that is needed to instanciate specific
- * components with given parameters.
+ * Initializes a created Babeltrace config object according to the
+ * command-line arguments found in argv.
  *
  * Return value is set to the appropriate exit code to use.
  */
@@ -2439,6 +2410,12 @@ int bt_config_init_from_args(struct bt_config *cfg, int argc, const char *argv[]
                goto error;
        }
 
+       ret = add_env_var_plugin_paths(cfg);
+       if (ret) {
+               printf_err("Cannot append plugin paths from BABELTRACE_PLUGIN_PATH\n");
+               goto error;
+       }
+
        /* Note: implicit source never gets positional base params. */
        implicit_source_comp = bt_config_component_from_arg(DEFAULT_SOURCE_COMPONENT_NAME);
        if (implicit_source_comp) {
@@ -2464,10 +2441,10 @@ int bt_config_init_from_args(struct bt_config *cfg, int argc, const char *argv[]
 
                switch (opt) {
                case OPT_PLUGIN_PATH:
-                       if (is_setuid_setgid()) {
-                               printf_debug("Skipping non-system plugin paths for setuid/setgid binary.");
+                       if (bt_common_is_setuid_setgid()) {
+                               printf_debug("Skipping non-system plugin paths for setuid/setgid binary\n");
                        } else {
-                               if (plugin_paths_from_arg(cfg->plugin_paths, arg)) {
+                               if (bt_config_append_plugin_paths(cfg->plugin_paths, arg)) {
                                        printf_err("Invalid --plugin-path option's argument\n");
                                        goto error;
                                }
@@ -2883,7 +2860,7 @@ int bt_config_init_from_args(struct bt_config *cfg, int argc, const char *argv[]
                }
        }
 
-       if (add_internal_plugin_paths(cfg)) {
+       if (append_home_and_system_plugin_paths(cfg)) {
                goto error;
        }
 
This page took 0.0266 seconds and 4 git commands to generate.