]> git.ipfire.org Git - thirdparty/util-linux.git/commit
libmount: fix mount -oloop=/dev/loopX regression
authorStanislav Brabec <sbrabec@suse.cz>
Thu, 12 Jan 2017 14:45:27 +0000 (15:45 +0100)
committerKarel Zak <kzak@redhat.com>
Mon, 16 Jan 2017 11:25:45 +0000 (12:25 +0100)
commit3789806db21dea8e009ceea8f6158bc778151afe
tree1ad67f76362ab89d06b32d539f753ab91bd320c1
parentb2140d2f24e046d57f9c653aad2f21732f04a51e
libmount: fix mount -oloop=/dev/loopX regression

If mount option "loop" is used with an argument, it should be respected. Commit
8efad715 introduced a regression. Even with an argument, overlaying loop device
is searched and argument is ignored. It could have unexpected side effects.

If argument is set, never allocate a new loop device.

How to reproduce:
mkdir -p cdrom
mkisofs -o cdrom.iso cdrom/ 2>/dev/null
losetup /dev/loop0 cdrom.iso
strace mount -t auto -o ro,loop=/dev/loop1 cdrom.iso /mnt 2>&1 | grep ^mount
cat /proc/self/mountinfo | grep /mnt
umount /mnt
losetup -d /dev/loop0

mount("/dev/loop0", "/mnt", "iso9660", MS_MGC_VAL|MS_RDONLY, NULL) = 0
327 60 7:0 / /mnt ro,relatime shared:241 - iso9660 /dev/loop0 ro
losetup: /dev/loop0: detach failed: No such device or address

Signed-off-by: Stanislav Brabec <sbrabec@suse.cz>
libmount/src/context_loopdev.c