]> git.ipfire.org Git - thirdparty/systemd.git/commitdiff
basic/namespace-util: fix double logging after fork failure 35440/head
authorZbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl>
Mon, 2 Dec 2024 10:25:53 +0000 (11:25 +0100)
committerZbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl>
Mon, 2 Dec 2024 10:51:23 +0000 (11:51 +0100)
[   10.056930] (journald)[104]: Failed to fork off '(sd-mkuserns)': Invalid argument
[   10.063727] systemd[1]: systemd-modules-load.service: About to execute: /usr/lib/systemd/systemd-modules-load
[   10.071148] (journald)[104]: Failed to fork process (sd-mkuserns): Invalid argument

safe_fork_full() already logs at debug level, so the caller shouldn't.

src/basic/namespace-util.c

index 5bc7ff6ab5f1d675d666de27cc48dc14122df52a..332e8cdfd5de64a58edef5ac7e426dee08a89a58 100644 (file)
@@ -348,7 +348,7 @@ int userns_acquire(const char *uid_map, const char *gid_map) {
 
         r = safe_fork("(sd-mkuserns)", FORK_CLOSE_ALL_FDS|FORK_DEATHSIG_SIGKILL|FORK_NEW_USERNS, &pid);
         if (r < 0)
-                return log_debug_errno(r, "Failed to fork process (sd-mkuserns): %m");
+                return r;
         if (r == 0)
                 /* Child. We do nothing here, just freeze until somebody kills us. */
                 freeze();