From 024f268d634159849e642e7005871739c428524d Mon Sep 17 00:00:00 2001 From: Lennart Poettering Date: Fri, 24 Aug 2012 22:50:16 +0200 Subject: [PATCH] shutdown: don't attempt read-only mounts in a container --- src/core/umount.c | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/src/core/umount.c b/src/core/umount.c index b9afac7ed..83c9de3e8 100644 --- a/src/core/umount.c +++ b/src/core/umount.c @@ -34,6 +34,7 @@ #include "umount.h" #include "path-util.h" #include "util.h" +#include "virt.h" typedef struct MountPoint { char *path; @@ -548,11 +549,9 @@ static int dm_points_list_detach(MountPoint **head, bool *changed) { int umount_all(bool *changed) { int r; bool umount_changed; - LIST_HEAD(MountPoint, mp_list_head); LIST_HEAD_INIT(MountPoint, mp_list_head); - r = mount_points_list_get(&mp_list_head); if (r < 0) goto end; @@ -572,7 +571,12 @@ int umount_all(bool *changed) { if (r <= 0) goto end; - r = mount_points_list_remount_read_only(&mp_list_head, changed); + /* If we are in a container, don't attempt to read-only mount + anything as that brings no real benefits, but might confuse + the host, as we remount the superblock here, not the bind + mound. */ + if (detect_container(NULL) <= 0) + r = mount_points_list_remount_read_only(&mp_list_head, changed); end: mount_points_list_free(&mp_list_head); -- 2.30.2