From: Lennart Poettering <lennart@poettering.net>
Date: Tue, 16 May 2023 09:44:27 +0000 (+0200)
Subject: base-filesystem: mention why we don't carry an entry for /tmp/ for now
X-Git-Tag: v254-rc1~470^2
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=refs%2Fpull%2F27658%2Fhead;p=thirdparty%2Fsystemd.git

base-filesystem: mention why we don't carry an entry for /tmp/ for now
---

diff --git a/src/shared/base-filesystem.c b/src/shared/base-filesystem.c
index 2d91e80ad60..0244c92c798 100644
--- a/src/shared/base-filesystem.c
+++ b/src/shared/base-filesystem.c
@@ -41,6 +41,10 @@ static const BaseFilesystem table[] = {
         { "sys",   0555, NULL,                         NULL, true },
         { "dev",   0555, NULL,                         NULL, true },
         { "run",   0555, NULL,                         NULL, true },
+        /* We don't add /tmp/ here for now (even though it's necessary for regular operation), because we
+         * want to support both cases where /tmp/ is a mount of its own (in which case we probably should set
+         * the mode to 1555, to indicate that noone should write to it, not even root) and when it's part of
+         * the rootfs (in which case we should set mode 1777), and we simply don't know what's right. */
 
         /* Various architecture ABIs define the path to the dynamic loader via the /lib64/ subdirectory of
          * the root directory. When booting from an otherwise empty root file system (where only /usr/ has