teknoraver / rpms / systemd

Forked from rpms/systemd 2 months ago
Clone

Blame 0030-escape-beef-up-new-systemd-escape-tool.patch

Zbigniew Jędrzejewski-Szmek 43ff24
From ee228789816679b6fff19c7c2f637eb0a1a3fcc4 Mon Sep 17 00:00:00 2001
Zbigniew Jędrzejewski-Szmek 43ff24
From: Lennart Poettering <lennart@poettering.net>
Zbigniew Jędrzejewski-Szmek 43ff24
Date: Mon, 7 Jul 2014 22:23:00 +0200
Zbigniew Jędrzejewski-Szmek 43ff24
Subject: [PATCH] escape: beef up new systemd-escape tool
Zbigniew Jędrzejewski-Szmek 43ff24
Zbigniew Jędrzejewski-Szmek 43ff24
Add various options for making it easy unescape, or mangle, or format as
Zbigniew Jędrzejewski-Szmek 43ff24
template instance or append a suffix.
Zbigniew Jędrzejewski-Szmek 43ff24
Zbigniew Jędrzejewski-Szmek 43ff24
(cherry picked from commit a1948c7bfeb87b54bc7715a44490c01593ee6e23)
Zbigniew Jędrzejewski-Szmek 43ff24
Zbigniew Jędrzejewski-Szmek 43ff24
Conflicts:
Zbigniew Jędrzejewski-Szmek 43ff24
	.gitignore
Zbigniew Jędrzejewski-Szmek 43ff24
---
Zbigniew Jędrzejewski-Szmek 43ff24
 .gitignore          |   1 +
Zbigniew Jędrzejewski-Szmek 43ff24
 src/escape/Makefile |   1 +
Zbigniew Jędrzejewski-Szmek 43ff24
 src/escape/escape.c | 215 +++++++++++++++++++++++++++++++++++++++++++++++++---
Zbigniew Jędrzejewski-Szmek 43ff24
 3 files changed, 206 insertions(+), 11 deletions(-)
Zbigniew Jędrzejewski-Szmek 43ff24
 create mode 120000 src/escape/Makefile
Zbigniew Jędrzejewski-Szmek 43ff24
Zbigniew Jędrzejewski-Szmek 43ff24
diff --git a/.gitignore b/.gitignore
Zbigniew Jędrzejewski-Szmek 43ff24
index 9523ea027e..e08aa52aee 100644
Zbigniew Jędrzejewski-Szmek 43ff24
--- a/.gitignore
Zbigniew Jędrzejewski-Szmek 43ff24
+++ b/.gitignore
Zbigniew Jędrzejewski-Szmek 43ff24
@@ -64,6 +64,7 @@
Zbigniew Jędrzejewski-Szmek 43ff24
 /systemd-delta
Zbigniew Jędrzejewski-Szmek 43ff24
 /systemd-detect-virt
Zbigniew Jędrzejewski-Szmek 43ff24
 /systemd-efi-boot-generator
Zbigniew Jędrzejewski-Szmek 43ff24
+/systemd-escape
Zbigniew Jędrzejewski-Szmek 43ff24
 /systemd-fsck
Zbigniew Jędrzejewski-Szmek 43ff24
 /systemd-fstab-generator
Zbigniew Jędrzejewski-Szmek 43ff24
 /systemd-getty-generator
Zbigniew Jędrzejewski-Szmek 43ff24
diff --git a/src/escape/Makefile b/src/escape/Makefile
Zbigniew Jędrzejewski-Szmek 43ff24
new file mode 120000
Zbigniew Jędrzejewski-Szmek 43ff24
index 0000000000..d0b0e8e008
Zbigniew Jędrzejewski-Szmek 43ff24
--- /dev/null
Zbigniew Jędrzejewski-Szmek 43ff24
+++ b/src/escape/Makefile
Zbigniew Jędrzejewski-Szmek 43ff24
@@ -0,0 +1 @@
Zbigniew Jędrzejewski-Szmek 43ff24
+../Makefile
Zbigniew Jędrzejewski-Szmek 43ff24
\ No newline at end of file
Zbigniew Jędrzejewski-Szmek 43ff24
diff --git a/src/escape/escape.c b/src/escape/escape.c
Zbigniew Jędrzejewski-Szmek 43ff24
index 0a59a05e28..ae0c183eca 100644
Zbigniew Jędrzejewski-Szmek 43ff24
--- a/src/escape/escape.c
Zbigniew Jędrzejewski-Szmek 43ff24
+++ b/src/escape/escape.c
Zbigniew Jędrzejewski-Szmek 43ff24
@@ -21,26 +21,219 @@
Zbigniew Jędrzejewski-Szmek 43ff24
 
Zbigniew Jędrzejewski-Szmek 43ff24
 #include <stdio.h>
Zbigniew Jędrzejewski-Szmek 43ff24
 #include <stdlib.h>
Zbigniew Jędrzejewski-Szmek 43ff24
+#include <getopt.h>
Zbigniew Jędrzejewski-Szmek 43ff24
 
Zbigniew Jędrzejewski-Szmek 43ff24
 #include "log.h"
Zbigniew Jędrzejewski-Szmek 43ff24
 #include "unit-name.h"
Zbigniew Jędrzejewski-Szmek 43ff24
+#include "build.h"
Zbigniew Jędrzejewski-Szmek 43ff24
+#include "strv.h"
Zbigniew Jędrzejewski-Szmek 43ff24
 
Zbigniew Jędrzejewski-Szmek 43ff24
-int main(int argc, char *argv[]) {
Zbigniew Jędrzejewski-Szmek 43ff24
-        char *escaped_name = NULL;
Zbigniew Jędrzejewski-Szmek 43ff24
+static enum {
Zbigniew Jędrzejewski-Szmek 43ff24
+        ACTION_ESCAPE,
Zbigniew Jędrzejewski-Szmek 43ff24
+        ACTION_UNESCAPE,
Zbigniew Jędrzejewski-Szmek 43ff24
+        ACTION_MANGLE
Zbigniew Jędrzejewski-Szmek 43ff24
+} arg_action = ACTION_ESCAPE;
Zbigniew Jędrzejewski-Szmek 43ff24
+static const char *arg_suffix = NULL;
Zbigniew Jędrzejewski-Szmek 43ff24
+static const char *arg_template = NULL;
Zbigniew Jędrzejewski-Szmek 43ff24
+static bool arg_path = false;
Zbigniew Jędrzejewski-Szmek 43ff24
+
Zbigniew Jędrzejewski-Szmek 43ff24
+static int help(void) {
Zbigniew Jędrzejewski-Szmek 43ff24
+
Zbigniew Jędrzejewski-Szmek 43ff24
+        printf("%s [OPTIONS...] [NAME...]\n\n"
Zbigniew Jędrzejewski-Szmek 43ff24
+               "Show system and user paths.\n\n"
Zbigniew Jędrzejewski-Szmek 43ff24
+               "  -h --help               Show this help\n"
Zbigniew Jędrzejewski-Szmek 43ff24
+               "     --version            Show package version\n"
Zbigniew Jędrzejewski-Szmek 43ff24
+               "     --suffix=SUFFIX      Unit suffix to append to escaped strings\n"
Zbigniew Jędrzejewski-Szmek 43ff24
+               "     --template=TEMPLATE  Insert strings as instance into template\n"
Zbigniew Jędrzejewski-Szmek 43ff24
+               "  -u --unescape           Unescape strings\n"
Zbigniew Jędrzejewski-Szmek 43ff24
+               "  -m --mangle             Mangle strings\n"
Zbigniew Jędrzejewski-Szmek 43ff24
+               "  -p --path               When escaping/unescaping assume the string is a path\n",
Zbigniew Jędrzejewski-Szmek 43ff24
+               program_invocation_short_name);
Zbigniew Jędrzejewski-Szmek 43ff24
+
Zbigniew Jędrzejewski-Szmek 43ff24
+        return 0;
Zbigniew Jędrzejewski-Szmek 43ff24
+}
Zbigniew Jędrzejewski-Szmek 43ff24
+
Zbigniew Jędrzejewski-Szmek 43ff24
+static int parse_argv(int argc, char *argv[]) {
Zbigniew Jędrzejewski-Szmek 43ff24
+
Zbigniew Jędrzejewski-Szmek 43ff24
+        enum {
Zbigniew Jędrzejewski-Szmek 43ff24
+                ARG_VERSION = 0x100,
Zbigniew Jędrzejewski-Szmek 43ff24
+                ARG_SUFFIX,
Zbigniew Jędrzejewski-Szmek 43ff24
+                ARG_TEMPLATE
Zbigniew Jędrzejewski-Szmek 43ff24
+        };
Zbigniew Jędrzejewski-Szmek 43ff24
+
Zbigniew Jędrzejewski-Szmek 43ff24
+        static const struct option options[] = {
Zbigniew Jędrzejewski-Szmek 43ff24
+                { "help",      no_argument,       NULL, 'h'           },
Zbigniew Jędrzejewski-Szmek 43ff24
+                { "version",   no_argument,       NULL, ARG_VERSION   },
Zbigniew Jędrzejewski-Szmek 43ff24
+                { "suffix",    required_argument, NULL, ARG_SUFFIX    },
Zbigniew Jędrzejewski-Szmek 43ff24
+                { "template",  required_argument, NULL, ARG_TEMPLATE  },
Zbigniew Jędrzejewski-Szmek 43ff24
+                { "unescape",  no_argument,       NULL, 'u'           },
Zbigniew Jędrzejewski-Szmek 43ff24
+                { "mangle",    no_argument,       NULL, 'm'           },
Zbigniew Jędrzejewski-Szmek 43ff24
+                { "path",      no_argument,       NULL, 'p'           },
Zbigniew Jędrzejewski-Szmek 43ff24
+                {}
Zbigniew Jędrzejewski-Szmek 43ff24
+        };
Zbigniew Jędrzejewski-Szmek 43ff24
+
Zbigniew Jędrzejewski-Szmek 43ff24
+        int c;
Zbigniew Jędrzejewski-Szmek 43ff24
+
Zbigniew Jędrzejewski-Szmek 43ff24
+        assert(argc >= 0);
Zbigniew Jędrzejewski-Szmek 43ff24
+        assert(argv);
Zbigniew Jędrzejewski-Szmek 43ff24
+
Zbigniew Jędrzejewski-Szmek 43ff24
+        while ((c = getopt_long(argc, argv, "hump", options, NULL)) >= 0) {
Zbigniew Jędrzejewski-Szmek 43ff24
+
Zbigniew Jędrzejewski-Szmek 43ff24
+                switch (c) {
Zbigniew Jędrzejewski-Szmek 43ff24
+
Zbigniew Jędrzejewski-Szmek 43ff24
+                case 'h':
Zbigniew Jędrzejewski-Szmek 43ff24
+                        return help();
Zbigniew Jędrzejewski-Szmek 43ff24
+
Zbigniew Jędrzejewski-Szmek 43ff24
+                case ARG_VERSION:
Zbigniew Jędrzejewski-Szmek 43ff24
+                        puts(PACKAGE_STRING);
Zbigniew Jędrzejewski-Szmek 43ff24
+                        puts(SYSTEMD_FEATURES);
Zbigniew Jędrzejewski-Szmek 43ff24
+                        return 0;
Zbigniew Jędrzejewski-Szmek 43ff24
+
Zbigniew Jędrzejewski-Szmek 43ff24
+                case ARG_SUFFIX:
Zbigniew Jędrzejewski-Szmek 43ff24
+
Zbigniew Jędrzejewski-Szmek 43ff24
+                        if (unit_type_from_string(optarg) < 0) {
Zbigniew Jędrzejewski-Szmek 43ff24
+                                log_error("Invalid unit suffix type %s.", optarg);
Zbigniew Jędrzejewski-Szmek 43ff24
+                                return -EINVAL;
Zbigniew Jędrzejewski-Szmek 43ff24
+                        }
Zbigniew Jędrzejewski-Szmek 43ff24
+
Zbigniew Jędrzejewski-Szmek 43ff24
+                        arg_suffix = optarg;
Zbigniew Jędrzejewski-Szmek 43ff24
+                        break;
Zbigniew Jędrzejewski-Szmek 43ff24
+
Zbigniew Jędrzejewski-Szmek 43ff24
+                case ARG_TEMPLATE:
Zbigniew Jędrzejewski-Szmek 43ff24
+
Zbigniew Jędrzejewski-Szmek 43ff24
+                        if (!unit_name_is_valid(optarg, true) || !unit_name_is_template(optarg)) {
Zbigniew Jędrzejewski-Szmek 43ff24
+                                log_error("Template name %s is not valid.", optarg);
Zbigniew Jędrzejewski-Szmek 43ff24
+                                return -EINVAL;
Zbigniew Jędrzejewski-Szmek 43ff24
+                        }
Zbigniew Jędrzejewski-Szmek 43ff24
+
Zbigniew Jędrzejewski-Szmek 43ff24
+                        arg_template = optarg;
Zbigniew Jędrzejewski-Szmek 43ff24
+                        break;
Zbigniew Jędrzejewski-Szmek 43ff24
+
Zbigniew Jędrzejewski-Szmek 43ff24
+                case 'u':
Zbigniew Jędrzejewski-Szmek 43ff24
+                        arg_action = ACTION_UNESCAPE;
Zbigniew Jędrzejewski-Szmek 43ff24
+                        break;
Zbigniew Jędrzejewski-Szmek 43ff24
+
Zbigniew Jędrzejewski-Szmek 43ff24
+                case 'm':
Zbigniew Jędrzejewski-Szmek 43ff24
+                        arg_action = ACTION_MANGLE;
Zbigniew Jędrzejewski-Szmek 43ff24
+                        break;
Zbigniew Jędrzejewski-Szmek 43ff24
+
Zbigniew Jędrzejewski-Szmek 43ff24
+                case 'p':
Zbigniew Jędrzejewski-Szmek 43ff24
+                        arg_path = true;
Zbigniew Jędrzejewski-Szmek 43ff24
+                        break;
Zbigniew Jędrzejewski-Szmek 43ff24
+
Zbigniew Jędrzejewski-Szmek 43ff24
+                case '?':
Zbigniew Jędrzejewski-Szmek 43ff24
+                        return -EINVAL;
Zbigniew Jędrzejewski-Szmek 43ff24
+
Zbigniew Jędrzejewski-Szmek 43ff24
+                default:
Zbigniew Jędrzejewski-Szmek 43ff24
+                        assert_not_reached("Unhandled option");
Zbigniew Jędrzejewski-Szmek 43ff24
+                }
Zbigniew Jędrzejewski-Szmek 43ff24
+        }
Zbigniew Jędrzejewski-Szmek 43ff24
 
Zbigniew Jędrzejewski-Szmek 43ff24
-        if (argc != 2) {
Zbigniew Jędrzejewski-Szmek 43ff24
-                log_error("This program requires on argument.");
Zbigniew Jędrzejewski-Szmek 43ff24
-                return EXIT_FAILURE;
Zbigniew Jędrzejewski-Szmek 43ff24
+        if (optind >= argc) {
Zbigniew Jędrzejewski-Szmek 43ff24
+                log_error("Not enough arguments.");
Zbigniew Jędrzejewski-Szmek 43ff24
+                return -EINVAL;
Zbigniew Jędrzejewski-Szmek 43ff24
         }
Zbigniew Jędrzejewski-Szmek 43ff24
 
Zbigniew Jędrzejewski-Szmek 43ff24
-        escaped_name = unit_name_escape(argv[1]);
Zbigniew Jędrzejewski-Szmek 43ff24
+        if (arg_template && arg_suffix) {
Zbigniew Jędrzejewski-Szmek 43ff24
+                log_error("--suffix= and --template= may not be combined.");
Zbigniew Jędrzejewski-Szmek 43ff24
+                return -EINVAL;
Zbigniew Jędrzejewski-Szmek 43ff24
+        }
Zbigniew Jędrzejewski-Szmek 43ff24
+
Zbigniew Jędrzejewski-Szmek 43ff24
+        if ((arg_template || arg_suffix) && arg_action != ACTION_ESCAPE) {
Zbigniew Jędrzejewski-Szmek 43ff24
+                log_error("--suffix= and --template= are not compatible with --unescape or --mangle.");
Zbigniew Jędrzejewski-Szmek 43ff24
+                return -EINVAL;
Zbigniew Jędrzejewski-Szmek 43ff24
+        }
Zbigniew Jędrzejewski-Szmek 43ff24
+
Zbigniew Jędrzejewski-Szmek 43ff24
+        if (arg_path && !IN_SET(arg_action, ACTION_ESCAPE, ACTION_UNESCAPE)) {
Zbigniew Jędrzejewski-Szmek 43ff24
+                log_error("--path may not be combined with --mangle.");
Zbigniew Jędrzejewski-Szmek 43ff24
+                return -EINVAL;
Zbigniew Jędrzejewski-Szmek 43ff24
+        }
Zbigniew Jędrzejewski-Szmek 43ff24
+
Zbigniew Jędrzejewski-Szmek 43ff24
+        return 1;
Zbigniew Jędrzejewski-Szmek 43ff24
+}
Zbigniew Jędrzejewski-Szmek 43ff24
+
Zbigniew Jędrzejewski-Szmek 43ff24
+int main(int argc, char *argv[]) {
Zbigniew Jędrzejewski-Szmek 43ff24
+        char **i;
Zbigniew Jędrzejewski-Szmek 43ff24
+        int r;
Zbigniew Jędrzejewski-Szmek 43ff24
+
Zbigniew Jędrzejewski-Szmek 43ff24
+        log_parse_environment();
Zbigniew Jędrzejewski-Szmek 43ff24
+        log_open();
Zbigniew Jędrzejewski-Szmek 43ff24
+
Zbigniew Jędrzejewski-Szmek 43ff24
+        r = parse_argv(argc, argv);
Zbigniew Jędrzejewski-Szmek 43ff24
+        if (r <= 0)
Zbigniew Jędrzejewski-Szmek 43ff24
+                goto finish;
Zbigniew Jędrzejewski-Szmek 43ff24
+
Zbigniew Jędrzejewski-Szmek 43ff24
+        STRV_FOREACH(i, argv + optind) {
Zbigniew Jędrzejewski-Szmek 43ff24
+                _cleanup_free_ char *e = NULL;
Zbigniew Jędrzejewski-Szmek 43ff24
+
Zbigniew Jędrzejewski-Szmek 43ff24
+                switch (arg_action) {
Zbigniew Jędrzejewski-Szmek 43ff24
+
Zbigniew Jędrzejewski-Szmek 43ff24
+                case ACTION_ESCAPE:
Zbigniew Jędrzejewski-Szmek 43ff24
+                        if (arg_path)
Zbigniew Jędrzejewski-Szmek 43ff24
+                                e = unit_name_path_escape(*i);
Zbigniew Jędrzejewski-Szmek 43ff24
+                        else
Zbigniew Jędrzejewski-Szmek 43ff24
+                                e = unit_name_escape(*i);
Zbigniew Jędrzejewski-Szmek 43ff24
+
Zbigniew Jędrzejewski-Szmek 43ff24
+                        if (!e) {
Zbigniew Jędrzejewski-Szmek 43ff24
+                                r = log_oom();
Zbigniew Jędrzejewski-Szmek 43ff24
+                                goto finish;
Zbigniew Jędrzejewski-Szmek 43ff24
+                        }
Zbigniew Jędrzejewski-Szmek 43ff24
+
Zbigniew Jędrzejewski-Szmek 43ff24
+                        if (arg_template) {
Zbigniew Jędrzejewski-Szmek 43ff24
+                                char *x;
Zbigniew Jędrzejewski-Szmek 43ff24
+
Zbigniew Jędrzejewski-Szmek 43ff24
+                                x = unit_name_replace_instance(arg_template, e);
Zbigniew Jędrzejewski-Szmek 43ff24
+                                if (!x) {
Zbigniew Jędrzejewski-Szmek 43ff24
+                                        r = log_oom();
Zbigniew Jędrzejewski-Szmek 43ff24
+                                        goto finish;
Zbigniew Jędrzejewski-Szmek 43ff24
+                                }
Zbigniew Jędrzejewski-Szmek 43ff24
+
Zbigniew Jędrzejewski-Szmek 43ff24
+                                free(e);
Zbigniew Jędrzejewski-Szmek 43ff24
+                                e = x;
Zbigniew Jędrzejewski-Szmek 43ff24
+                        } else if (arg_suffix) {
Zbigniew Jędrzejewski-Szmek 43ff24
+                                char *x;
Zbigniew Jędrzejewski-Szmek 43ff24
+
Zbigniew Jędrzejewski-Szmek 43ff24
+                                x = strjoin(e, ".", arg_suffix, NULL);
Zbigniew Jędrzejewski-Szmek 43ff24
+                                if (!x) {
Zbigniew Jędrzejewski-Szmek 43ff24
+                                        r = log_oom();
Zbigniew Jędrzejewski-Szmek 43ff24
+                                        goto finish;
Zbigniew Jędrzejewski-Szmek 43ff24
+                                }
Zbigniew Jędrzejewski-Szmek 43ff24
+
Zbigniew Jędrzejewski-Szmek 43ff24
+                                free(e);
Zbigniew Jędrzejewski-Szmek 43ff24
+                                e = x;
Zbigniew Jędrzejewski-Szmek 43ff24
+                        }
Zbigniew Jędrzejewski-Szmek 43ff24
+
Zbigniew Jędrzejewski-Szmek 43ff24
+                        break;
Zbigniew Jędrzejewski-Szmek 43ff24
+
Zbigniew Jędrzejewski-Szmek 43ff24
+                case ACTION_UNESCAPE:
Zbigniew Jędrzejewski-Szmek 43ff24
+                        if (arg_path)
Zbigniew Jędrzejewski-Szmek 43ff24
+                                e = unit_name_path_unescape(*i);
Zbigniew Jędrzejewski-Szmek 43ff24
+                        else
Zbigniew Jędrzejewski-Szmek 43ff24
+                                e = unit_name_unescape(*i);
Zbigniew Jędrzejewski-Szmek 43ff24
+
Zbigniew Jędrzejewski-Szmek 43ff24
+                        if (!e) {
Zbigniew Jędrzejewski-Szmek 43ff24
+                                r = log_oom();
Zbigniew Jędrzejewski-Szmek 43ff24
+                                goto finish;
Zbigniew Jędrzejewski-Szmek 43ff24
+                        }
Zbigniew Jędrzejewski-Szmek 43ff24
+                        break;
Zbigniew Jędrzejewski-Szmek 43ff24
+
Zbigniew Jędrzejewski-Szmek 43ff24
+                case ACTION_MANGLE:
Zbigniew Jędrzejewski-Szmek 43ff24
+                        e = unit_name_mangle(*i, MANGLE_NOGLOB);
Zbigniew Jędrzejewski-Szmek 43ff24
+                        if (!e) {
Zbigniew Jędrzejewski-Szmek 43ff24
+                                r = log_oom();
Zbigniew Jędrzejewski-Szmek 43ff24
+                                goto finish;
Zbigniew Jędrzejewski-Szmek 43ff24
+                        }
Zbigniew Jędrzejewski-Szmek 43ff24
+                        break;
Zbigniew Jędrzejewski-Szmek 43ff24
+                }
Zbigniew Jędrzejewski-Szmek 43ff24
+
Zbigniew Jędrzejewski-Szmek 43ff24
+                if (i != argv+optind)
Zbigniew Jędrzejewski-Szmek 43ff24
+                        fputc(' ', stdout);
Zbigniew Jędrzejewski-Szmek 43ff24
 
Zbigniew Jędrzejewski-Szmek 43ff24
-        if (!escaped_name) {
Zbigniew Jędrzejewski-Szmek 43ff24
-                log_error("Failed to escape name.");
Zbigniew Jędrzejewski-Szmek 43ff24
-                return EXIT_FAILURE;
Zbigniew Jędrzejewski-Szmek 43ff24
+                fputs(e, stdout);
Zbigniew Jędrzejewski-Szmek 43ff24
         }
Zbigniew Jędrzejewski-Szmek 43ff24
 
Zbigniew Jędrzejewski-Szmek 43ff24
-        printf("%s", escaped_name);
Zbigniew Jędrzejewski-Szmek 43ff24
+        fputc('\n', stdout);
Zbigniew Jędrzejewski-Szmek 43ff24
 
Zbigniew Jędrzejewski-Szmek 43ff24
-        return EXIT_SUCCESS;
Zbigniew Jędrzejewski-Szmek 43ff24
+finish:
Zbigniew Jędrzejewski-Szmek 43ff24
+        return r ? EXIT_FAILURE : EXIT_SUCCESS;
Zbigniew Jędrzejewski-Szmek 43ff24
 }