]> git.ipfire.org Git - thirdparty/systemd.git/commitdiff
basic/umask-util: drop clearing of unused bits in call to umask(2)
authorZbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl>
Mon, 21 Aug 2023 18:44:05 +0000 (20:44 +0200)
committerZbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl>
Tue, 22 Aug 2023 10:31:45 +0000 (12:31 +0200)
umask(2) says that the kernel does this exact operation on its own, so
we can drop if from our call.

src/basic/umask-util.h

index 6f0e1cc2b24a88328da71b89a9d2f74b11e4801e..00417fa3045c8a9b0aec9cf69cc304a7231ffaee 100644 (file)
@@ -8,12 +8,12 @@
 #include "macro.h"
 
 static inline void umaskp(mode_t *u) {
-        umask(*u & 0777);
+        umask(*u);
 }
 
 #define _cleanup_umask_ _cleanup_(umaskp)
 
-/* We make use of the fact here that the umask() concept is using only the lower 9 bits of mode_t, although
+/* We make use of the fact here that the umask() syscall uses only the lower 9 bits of mode_t, although
  * mode_t has space for the file type in the bits further up. We simply OR in the file type mask S_IFMT to
  * distinguish the first and the second iteration of the WITH_UMASK() loop, so that we can run the first one,
  * and exit on the second. */