chiark / gitweb /
memfd: always create our memfds with CLOEXEC set
authorLennart Poettering <lennart@poettering.net>
Thu, 30 Oct 2014 15:23:34 +0000 (16:23 +0100)
committerLennart Poettering <lennart@poettering.net>
Thu, 30 Oct 2014 15:23:34 +0000 (16:23 +0100)
We really shouldn't create fds ever that have the flag unset.

src/libsystemd/sd-bus/bus-kernel.c
src/shared/memfd.c
src/shared/missing.h

index 0327614..3f8b0ab 100644 (file)
@@ -1129,7 +1129,7 @@ int bus_kernel_pop_memfd(sd_bus *bus, void **address, size_t *mapped, size_t *al
                 if (!g)
                         return -ENOMEM;
 
                 if (!g)
                         return -ENOMEM;
 
-                r = memfd_create(g, MFD_ALLOW_SEALING);
+                r = memfd_create(g, MFD_ALLOW_SEALING|MFD_CLOEXEC);
                 if (r < 0)
                         return -errno;
 
                 if (r < 0)
                         return -errno;
 
index 162c12f..f3ce8f8 100644 (file)
@@ -65,7 +65,7 @@ int memfd_new(const char *name) {
                 }
         }
 
                 }
         }
 
-        fd = memfd_create(name, MFD_ALLOW_SEALING);
+        fd = memfd_create(name, MFD_ALLOW_SEALING | MFD_CLOEXEC);
         if (fd < 0)
                 return -errno;
 
         if (fd < 0)
                 return -errno;
 
index 7725e47..0d7c559 100644 (file)
 #endif
 
 #ifndef MFD_ALLOW_SEALING
 #endif
 
 #ifndef MFD_ALLOW_SEALING
-#define MFD_ALLOW_SEALING 0x0002ULL
+#define MFD_ALLOW_SEALING 0x0002U
+#endif
+
+#ifndef MFD_CLOEXEC
+#define MFD_CLOEXEC 0x0001U
 #endif
 
 #ifndef IP_FREEBIND
 #endif
 
 #ifndef IP_FREEBIND