]> xenbits.xensource.com Git - libvirt.git/commit
qemu_namespace: Deal with nested mounts when umount()-ing /dev
authorMichal Privoznik <mprivozn@redhat.com>
Tue, 7 Feb 2023 14:06:32 +0000 (15:06 +0100)
committerMichal Privoznik <mprivozn@redhat.com>
Wed, 8 Feb 2023 07:39:17 +0000 (08:39 +0100)
commit5155ab4b2a704285505dfea6ffee8b980fdaa29e
tree64b24ad3afb92b9706d23ada43f4350c5dc1212f
parent697c16e39ae9a9e18ce7cad0729bf2293b12a307
qemu_namespace: Deal with nested mounts when umount()-ing /dev

In one of recent commits (v9.0.0-rc1~106) I've made our QEMU
namespace code umount the original /dev. One of the reasons was
enhanced security, because previously we just mounted a tmpfs
over the original /dev. Thus a malicious QEMU could just
umount("/dev") and it would get to the original /dev with all
nodes.

Now, on some systems this introduced a regression:

   failed to umount devfs on /dev: Device or resource busy

But how this could be? We've moved all file systems mounted under
/dev to a temporary location. Or have we? As it turns out, not
quite. If there are two file systems mounted on the same target,
e.g. like this:

  mount -t tmpfs tmpfs /dev/shm/ && mount -t tmpfs tmpfs /dev/shm/

then only the top most (i.e. the last one) is moved. See
qemuDomainUnshareNamespace() for more info.

Now, we could enhance our code to deal with these "doubled" mount
points. Or, since it is the top most file system that is
accessible anyways (and this one is preserved), we can
umount("/dev") in a recursive fashion.

Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=2167302
Fixes: 379c0ce4bfed8733dfbde557c359eecc5474ce38
Signed-off-by: Michal Privoznik <mprivozn@redhat.com>
Reviewed-by: Jim Fehlig <jfehlig@suse.com>
src/qemu/qemu_namespace.c