|
 |
17b0f1 |
From 8d166597076d87aae9d5f98144103386c79d6446 Mon Sep 17 00:00:00 2001
|
|
|
17b0f1 |
From: Jan Synacek <jsynacek@redhat.com>
|
|
|
17b0f1 |
Date: Fri, 31 Mar 2017 09:47:46 +0200
|
|
|
17b0f1 |
Subject: [PATCH] namespace: don't fail on masked mounts
|
|
|
17b0f1 |
|
|
|
17b0f1 |
Before this patch, a service file with ReadWriteDirectories=/file...
|
|
|
17b0f1 |
could fail if the file exists but is not a mountpoint, despite being
|
|
|
17b0f1 |
listed in /proc/self/mountinfo. It could happen with masked mounts.
|
|
|
17b0f1 |
|
|
|
17b0f1 |
(cherry picked from commit 98df8089bea1b2407c46495b6c2eb76dda46c658)
|
|
|
17b0f1 |
|
|
|
17b0f1 |
Resolves: #1433687
|
|
|
17b0f1 |
---
|
|
|
17b0f1 |
src/shared/util.c | 23 +++++++++++------------
|
|
|
17b0f1 |
1 file changed, 11 insertions(+), 12 deletions(-)
|
|
|
17b0f1 |
|
|
|
17b0f1 |
diff --git a/src/shared/util.c b/src/shared/util.c
|
|
|
17b0f1 |
index 1070e32c4a..3e13cc1fdb 100644
|
|
|
17b0f1 |
--- a/src/shared/util.c
|
|
|
17b0f1 |
+++ b/src/shared/util.c
|
|
|
17b0f1 |
@@ -7332,22 +7332,21 @@ int bind_remount_recursive(const char *prefix, bool ro) {
|
|
|
17b0f1 |
if (r < 0)
|
|
|
17b0f1 |
return r;
|
|
|
17b0f1 |
|
|
|
17b0f1 |
- /* Try to reuse the original flag set, but
|
|
|
17b0f1 |
- * don't care for errors, in case of
|
|
|
17b0f1 |
- * obstructed mounts */
|
|
|
17b0f1 |
+ /* Deal with mount points that are obstructed by a
|
|
|
17b0f1 |
+ * later mount */
|
|
|
17b0f1 |
+ r = path_is_mount_point(x, 0);
|
|
|
17b0f1 |
+ if (r == -ENOENT || r == 0)
|
|
|
17b0f1 |
+ continue;
|
|
|
17b0f1 |
+ if (r < 0)
|
|
|
17b0f1 |
+ return r;
|
|
|
17b0f1 |
+
|
|
|
17b0f1 |
+ /* Try to reuse the original flag set */
|
|
|
17b0f1 |
orig_flags = 0;
|
|
|
17b0f1 |
(void) get_mount_flags(x, &orig_flags);
|
|
|
17b0f1 |
orig_flags &= ~MS_RDONLY;
|
|
|
17b0f1 |
|
|
|
17b0f1 |
- if (mount(NULL, x, NULL, orig_flags|MS_BIND|MS_REMOUNT|(ro ? MS_RDONLY : 0), NULL) < 0) {
|
|
|
17b0f1 |
-
|
|
|
17b0f1 |
- /* Deal with mount points that are
|
|
|
17b0f1 |
- * obstructed by a later mount */
|
|
|
17b0f1 |
-
|
|
|
17b0f1 |
- if (errno != ENOENT)
|
|
|
17b0f1 |
- return -errno;
|
|
|
17b0f1 |
- }
|
|
|
17b0f1 |
-
|
|
|
17b0f1 |
+ if (mount(NULL, x, NULL, orig_flags|MS_BIND|MS_REMOUNT|(ro ? MS_RDONLY : 0), NULL) < 0)
|
|
|
17b0f1 |
+ return -errno;
|
|
|
17b0f1 |
}
|
|
|
17b0f1 |
}
|
|
|
17b0f1 |
}
|