Re: [PATCH RFC 0/1] mount: universally disallow mounting over symlinks

From
Aleksa Sarai <hidden>
Date
2020-01-02 03:59:42
Also in
linux-fsdevel, lkml, stable
Reply to
Al Viro, 2020-01-01
On 2020-01-01, Al Viro [off-list ref] wrote:
On Thu, Jan 02, 2020 at 01:44:07AM +1100, Aleksa Sarai wrote:
quoted
Thanks, this fixes the issue for me (and also fixes another reproducer I
found -- mounting a symlink on top of itself then trying to umount it).

Reported-by: Aleksa Sarai <redacted>
Tested-by: Aleksa Sarai <redacted>
Pushed into #fixes.
Thanks. One other thing I noticed is that umount applies to the
underlying symlink rather than the mountpoint on top. So, for example
(using the same scripts I posted in the thread):

  # ln -s /tmp/foo link
  # ./mount_to_symlink /etc/passwd link
  # umount -l link # will attempt to unmount "/tmp/foo"

Is that intentional?

-- 
Aleksa Sarai
Senior Software Engineer (Containers)
SUSE Linux GmbH
<https://www.cyphar.com/>

Attachments

Thread (60 messages) 60 messages, 7 authors, last reply 2020-01-19
Keyboard shortcuts
hback out one level
jnext message in thread / next series-diff position
kprevious message in thread / previous series-diff position
ldrill in
Escclose help / fold thread tree
?toggle this help