X-Git-Url: http://git.ipfire.org/?a=blobdiff_plain;f=src%2Fbinfmt%2Fbinfmt.c;h=43ed2f385bf25220f6e94640e7fd08e1b7a908ea;hb=db9ecf050165fd1033c6f81485917e229c4be537;hp=0eb9e4601462817bd0106a180c0454303161c84a;hpb=a826d4f7d01b144df191cf7a6de729b678cdb1d9;p=thirdparty%2Fsystemd.git diff --git a/src/binfmt/binfmt.c b/src/binfmt/binfmt.c index 0eb9e460146..43ed2f385bf 100644 --- a/src/binfmt/binfmt.c +++ b/src/binfmt/binfmt.c @@ -1,9 +1,4 @@ -/* SPDX-License-Identifier: LGPL-2.1+ */ -/*** - This file is part of systemd. - - Copyright 2010 Lennart Poettering -***/ +/* SPDX-License-Identifier: LGPL-2.1-or-later */ #include #include @@ -11,47 +6,57 @@ #include #include #include -#include +#include +#include #include "alloc-util.h" +#include "binfmt-util.h" #include "conf-files.h" #include "def.h" #include "fd-util.h" #include "fileio.h" #include "log.h" +#include "main-func.h" +#include "pager.h" +#include "path-util.h" +#include "pretty-print.h" #include "string-util.h" #include "strv.h" -#include "terminal-util.h" -#include "util.h" static bool arg_cat_config = false; +static PagerFlags arg_pager_flags = 0; +static bool arg_unregister = false; static int delete_rule(const char *rule) { _cleanup_free_ char *x = NULL, *fn = NULL; char *e; + assert(rule); assert(rule[0]); - x = strdup(rule); + e = strchrnul(rule + 1, rule[0]); + x = strndup(rule + 1, e - rule - 1); if (!x) return log_oom(); - e = strchrnul(x+1, x[0]); - *e = 0; + if (!filename_is_valid(x) || + STR_IN_SET(x, "register", "status")) + return log_error_errno(SYNTHETIC_ERRNO(EINVAL), + "Rule file name '%s' is not valid, refusing.", x); - fn = strappend("/proc/sys/fs/binfmt_misc/", x+1); + fn = path_join("/proc/sys/fs/binfmt_misc", x); if (!fn) return log_oom(); - return write_string_file(fn, "-1", 0); + return write_string_file(fn, "-1", WRITE_STRING_FILE_DISABLE_BUFFER); } static int apply_rule(const char *rule) { int r; - delete_rule(rule); + (void) delete_rule(rule); - r = write_string_file("/proc/sys/fs/binfmt_misc/register", rule, 0); + r = write_string_file("/proc/sys/fs/binfmt_misc/register", rule, WRITE_STRING_FILE_DISABLE_BUFFER); if (r < 0) return log_error_errno(r, "Failed to add binary format: %m"); @@ -69,25 +74,25 @@ static int apply_file(const char *path, bool ignore_enoent) { if (ignore_enoent && r == -ENOENT) return 0; - return log_error_errno(r, "Failed to open file '%s', ignoring: %m", path); + return log_error_errno(r, "Failed to open file '%s': %m", path); } log_debug("apply: %s", path); for (;;) { - char l[LINE_MAX], *p; + _cleanup_free_ char *line = NULL; + char *p; int k; - if (!fgets(l, sizeof(l), f)) { - if (feof(f)) - break; - - return log_error_errno(errno, "Failed to read file '%s', ignoring: %m", path); - } + k = read_line(f, LONG_LINE_MAX, &line); + if (k < 0) + return log_error_errno(k, "Failed to read file '%s': %m", path); + if (k == 0) + break; - p = strstrip(l); - if (!*p) + p = strstrip(line); + if (isempty(p)) continue; - if (strchr(COMMENTS "\n", *p)) + if (strchr(COMMENTS, p[0])) continue; k = apply_rule(p); @@ -98,26 +103,43 @@ static int apply_file(const char *path, bool ignore_enoent) { return r; } -static void help(void) { +static int help(void) { + _cleanup_free_ char *link = NULL; + int r; + + r = terminal_urlify_man("systemd-binfmt.service", "8", &link); + if (r < 0) + return log_oom(); + printf("%s [OPTIONS...] [CONFIGURATION FILE...]\n\n" - "Registers binary formats.\n\n" + "Registers binary formats with the kernel.\n\n" " -h --help Show this help\n" " --version Show package version\n" " --cat-config Show configuration files\n" - , program_invocation_short_name); + " --no-pager Do not pipe output into a pager\n" + " --unregister Unregister all existing entries\n" + "\nSee the %s for details.\n" + , program_invocation_short_name + , link + ); + + return 0; } static int parse_argv(int argc, char *argv[]) { - enum { ARG_VERSION = 0x100, ARG_CAT_CONFIG, + ARG_NO_PAGER, + ARG_UNREGISTER, }; static const struct option options[] = { - { "help", no_argument, NULL, 'h' }, - { "version", no_argument, NULL, ARG_VERSION }, - { "cat-config", no_argument, NULL, ARG_CAT_CONFIG }, + { "help", no_argument, NULL, 'h' }, + { "version", no_argument, NULL, ARG_VERSION }, + { "cat-config", no_argument, NULL, ARG_CAT_CONFIG }, + { "no-pager", no_argument, NULL, ARG_NO_PAGER }, + { "unregister", no_argument, NULL, ARG_UNREGISTER }, {} }; @@ -131,8 +153,7 @@ static int parse_argv(int argc, char *argv[]) { switch (c) { case 'h': - help(); - return 0; + return help(); case ARG_VERSION: return version(); @@ -141,6 +162,14 @@ static int parse_argv(int argc, char *argv[]) { arg_cat_config = true; break; + case ARG_NO_PAGER: + arg_pager_flags |= PAGER_DISABLE; + break; + + case ARG_UNREGISTER: + arg_unregister = true; + break; + case '?': return -EINVAL; @@ -148,29 +177,29 @@ static int parse_argv(int argc, char *argv[]) { assert_not_reached("Unhandled option"); } - if (arg_cat_config && argc > optind) { - log_error("Positional arguments are not allowed with --cat-config"); - return -EINVAL; - } + if ((arg_unregister || arg_cat_config) && argc > optind) + return log_error_errno(SYNTHETIC_ERRNO(EINVAL), + "Positional arguments are not allowed with --cat-config or --unregister"); return 1; } -int main(int argc, char *argv[]) { +static int run(int argc, char *argv[]) { int r, k; r = parse_argv(argc, argv); if (r <= 0) return r < 0 ? EXIT_FAILURE : EXIT_SUCCESS; - log_set_target(LOG_TARGET_AUTO); - log_parse_environment(); - log_open(); + log_setup_service(); umask(0022); r = 0; + if (arg_unregister) + return disable_binfmt(); + if (argc > optind) { int i; @@ -184,18 +213,17 @@ int main(int argc, char *argv[]) { char **f; r = conf_files_list_strv(&files, ".conf", NULL, 0, (const char**) CONF_PATHS_STRV("binfmt.d")); - if (r < 0) { - log_error_errno(r, "Failed to enumerate binfmt.d files: %m"); - goto finish; - } + if (r < 0) + return log_error_errno(r, "Failed to enumerate binfmt.d files: %m"); if (arg_cat_config) { - r = cat_files(NULL, files, 0); - goto finish; + (void) pager_open(arg_pager_flags); + + return cat_files(NULL, files, 0); } /* Flush out all rules */ - write_string_file("/proc/sys/fs/binfmt_misc/status", "-1", 0); + (void) write_string_file("/proc/sys/fs/binfmt_misc/status", "-1", WRITE_STRING_FILE_DISABLE_BUFFER); STRV_FOREACH(f, files) { k = apply_file(*f, true); @@ -204,6 +232,7 @@ int main(int argc, char *argv[]) { } } -finish: - return r < 0 ? EXIT_FAILURE : EXIT_SUCCESS; + return r; } + +DEFINE_MAIN_FUNCTION(run);