Blame SOURCES/0441-basic-fix-touch-creating-files-with-07777-mode.patch
|
 |
923a60 |
From 616db6ddcacd25e4c3a771cd317373971c9055ed Mon Sep 17 00:00:00 2001
|
|
|
923a60 |
From: =?UTF-8?q?Mantas=20Mikul=C4=97nas?= <grawity@gmail.com>
|
|
|
923a60 |
Date: Fri, 29 Jan 2016 23:36:08 +0200
|
|
|
923a60 |
Subject: [PATCH] basic: fix touch() creating files with 07777 mode
|
|
|
923a60 |
|
|
|
923a60 |
mode_t is unsigned, so MODE_INVALID < 0 can never be true.
|
|
|
923a60 |
|
|
|
923a60 |
This fixes a possible DoS where any user could fill /run by writing to
|
|
|
923a60 |
a world-writable /run/systemd/show-status.
|
|
|
923a60 |
|
|
|
923a60 |
Cherry-picked from: 06eeacb6fe029804f296b065b3ce91e796e1cd0e
|
|
|
923a60 |
Resolves: #1416062
|
|
|
923a60 |
---
|
|
|
923a60 |
src/shared/util.c | 3 ++-
|
|
|
923a60 |
1 file changed, 2 insertions(+), 1 deletion(-)
|
|
|
923a60 |
|
|
|
923a60 |
diff --git a/src/shared/util.c b/src/shared/util.c
|
|
|
923a60 |
index 66729f70e5..1070e32c4a 100644
|
|
|
923a60 |
--- a/src/shared/util.c
|
|
|
923a60 |
+++ b/src/shared/util.c
|
|
|
923a60 |
@@ -3908,7 +3908,8 @@ int touch_file(const char *path, bool parents, usec_t stamp, uid_t uid, gid_t gi
|
|
|
923a60 |
if (parents)
|
|
|
923a60 |
mkdir_parents(path, 0755);
|
|
|
923a60 |
|
|
|
923a60 |
- fd = open(path, O_WRONLY|O_CREAT|O_CLOEXEC|O_NOCTTY, mode > 0 ? mode : 0644);
|
|
|
923a60 |
+ fd = open(path, O_WRONLY|O_CREAT|O_CLOEXEC|O_NOCTTY,
|
|
|
923a60 |
+ (mode == 0 || mode == MODE_INVALID) ? 0644 : mode);
|
|
|
923a60 |
if (fd < 0)
|
|
|
923a60 |
return -errno;
|
|
|
923a60 |
|