From: Zbigniew Jędrzejewski-Szmek Date: Wed, 21 Nov 2018 09:12:00 +0000 (+0100) Subject: generators: drop umask calls X-Git-Tag: v240~255 X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=053254e3cb215df3b8c905bc39b920f8817e1c7d;p=thirdparty%2Fsystemd.git generators: drop umask calls systemd already sets the umask (see e3b8d0637dd755b3426f3363b2cdad63f738116c). When running under systemd, we don't need to set it. And when *not* running under systemd, for example during development, there is no reason to override the user config. Let's just drop those calls. $ git grep -e 'umask\(' -l 'src/*generator*' |xargs perl -i -0pe 's|^[^\n]*umask\([^\n]+\n\n||gms' --- diff --git a/src/boot/bless-boot-generator.c b/src/boot/bless-boot-generator.c index 0fd9c76d191..e28cccd7611 100644 --- a/src/boot/bless-boot-generator.c +++ b/src/boot/bless-boot-generator.c @@ -24,8 +24,6 @@ int main(int argc, char *argv[]) { log_setup_generator(); - umask(0022); - if (argc > 1 && argc != 4) { log_error("This program takes three or no arguments."); return EXIT_FAILURE; diff --git a/src/cryptsetup/cryptsetup-generator.c b/src/cryptsetup/cryptsetup-generator.c index 4ee77de8c32..15bfe9348f4 100644 --- a/src/cryptsetup/cryptsetup-generator.c +++ b/src/cryptsetup/cryptsetup-generator.c @@ -585,8 +585,6 @@ int main(int argc, char *argv[]) { log_setup_generator(); - umask(0022); - arg_disks = hashmap_new(&string_hash_ops); if (!arg_disks) { r = log_oom(); diff --git a/src/debug-generator/debug-generator.c b/src/debug-generator/debug-generator.c index d90f2321a03..6e30aa475c5 100644 --- a/src/debug-generator/debug-generator.c +++ b/src/debug-generator/debug-generator.c @@ -155,8 +155,6 @@ static int run(int argc, char *argv[]) { log_setup_generator(); - umask(0022); - r = proc_cmdline_parse(parse_proc_cmdline_item, NULL, PROC_CMDLINE_RD_STRICT | PROC_CMDLINE_STRIP_RD_PREFIX); if (r < 0) log_warning_errno(r, "Failed to parse kernel command line, ignoring: %m"); diff --git a/src/fstab-generator/fstab-generator.c b/src/fstab-generator/fstab-generator.c index 5098336abed..15ac20dd713 100644 --- a/src/fstab-generator/fstab-generator.c +++ b/src/fstab-generator/fstab-generator.c @@ -882,8 +882,6 @@ static int run(int argc, char *argv[]) { log_setup_generator(); - umask(0022); - r = proc_cmdline_parse(parse_proc_cmdline_item, NULL, 0); if (r < 0) log_warning_errno(r, "Failed to parse kernel command line, ignoring: %m"); diff --git a/src/getty-generator/getty-generator.c b/src/getty-generator/getty-generator.c index 80151546791..e2e07bca76f 100644 --- a/src/getty-generator/getty-generator.c +++ b/src/getty-generator/getty-generator.c @@ -123,8 +123,6 @@ int main(int argc, char *argv[]) { log_setup_generator(); - umask(0022); - if (detect_container() > 0) { _cleanup_free_ char *container_ttys = NULL; diff --git a/src/gpt-auto-generator/gpt-auto-generator.c b/src/gpt-auto-generator/gpt-auto-generator.c index 03942e94ea5..1ca071e9991 100644 --- a/src/gpt-auto-generator/gpt-auto-generator.c +++ b/src/gpt-auto-generator/gpt-auto-generator.c @@ -681,8 +681,6 @@ int main(int argc, char *argv[]) { log_setup_generator(); - umask(0022); - if (detect_container() > 0) { log_debug("In a container, exiting."); return EXIT_SUCCESS; diff --git a/src/hibernate-resume/hibernate-resume-generator.c b/src/hibernate-resume/hibernate-resume-generator.c index 64c184558f1..e225424ce23 100644 --- a/src/hibernate-resume/hibernate-resume-generator.c +++ b/src/hibernate-resume/hibernate-resume-generator.c @@ -74,8 +74,6 @@ static int run(int argc, char *argv[]) { log_setup_generator(); - umask(0022); - if (argc > 1 && argc != 4) { log_error("This program takes three or no arguments."); return -EINVAL; diff --git a/src/rc-local-generator/rc-local-generator.c b/src/rc-local-generator/rc-local-generator.c index 9135aca07e2..a8dc68bccca 100644 --- a/src/rc-local-generator/rc-local-generator.c +++ b/src/rc-local-generator/rc-local-generator.c @@ -54,8 +54,6 @@ int main(int argc, char *argv[]) { log_setup_generator(); - umask(0022); - if (access(RC_LOCAL_SCRIPT_PATH_START, X_OK) >= 0) { log_debug("Automatically adding rc-local.service."); diff --git a/src/system-update-generator/system-update-generator.c b/src/system-update-generator/system-update-generator.c index 4b7d7d58cde..06062e45100 100644 --- a/src/system-update-generator/system-update-generator.c +++ b/src/system-update-generator/system-update-generator.c @@ -64,8 +64,6 @@ int main(int argc, char *argv[]) { log_setup_generator(); - umask(0022); - r = generate_symlink(); if (r > 0) { diff --git a/src/sysv-generator/sysv-generator.c b/src/sysv-generator/sysv-generator.c index 6a1badabd49..c4acdc4a50b 100644 --- a/src/sysv-generator/sysv-generator.c +++ b/src/sysv-generator/sysv-generator.c @@ -934,8 +934,6 @@ static int run(int argc, char *argv[]) { if (argc > 1) arg_dest = argv[3]; - umask(0022); - r = lookup_paths_init(&lp, UNIT_FILE_SYSTEM, LOOKUP_PATHS_EXCLUDE_GENERATED, NULL); if (r < 0) return log_error_errno(r, "Failed to find lookup paths: %m"); diff --git a/src/veritysetup/veritysetup-generator.c b/src/veritysetup/veritysetup-generator.c index 5eaba5d32e1..33d803cd074 100644 --- a/src/veritysetup/veritysetup-generator.c +++ b/src/veritysetup/veritysetup-generator.c @@ -212,8 +212,6 @@ int main(int argc, char *argv[]) { log_setup_generator(); - umask(0022); - r = proc_cmdline_parse(parse_proc_cmdline_item, NULL, PROC_CMDLINE_STRIP_RD_PREFIX); if (r < 0) { log_warning_errno(r, "Failed to parse kernel command line: %m");