ovl: don't follow redirects if redirect_dir=off
authorMiklos Szeredi <mszeredi@redhat.com>
Mon, 11 Dec 2017 10:28:10 +0000 (11:28 +0100)
committerMiklos Szeredi <mszeredi@redhat.com>
Mon, 11 Dec 2017 10:28:10 +0000 (11:28 +0100)
commit438c84c2f0c794f75ab55ce65c505b01bfce4480
tree1473170ca270435dc5165c04d6690b7bcd6718bb
parentae64f9bd1d3621b5e60d7363bc20afb46aede215
ovl: don't follow redirects if redirect_dir=off

Overlayfs is following redirects even when redirects are disabled. If this
is unintentional (probably the majority of cases) then this can be a
problem.  E.g. upper layer comes from untrusted USB drive, and attacker
crafts a redirect to enable read access to otherwise unreadable
directories.

If "redirect_dir=off", then turn off following as well as creation of
redirects.  If "redirect_dir=follow", then turn on following, but turn off
creation of redirects (which is what "redirect_dir=off" does now).

This is a backward incompatible change, so make it dependent on a config
option.

Reported-by: David Howells <dhowells@redhat.com>
Signed-off-by: Miklos Szeredi <mszeredi@redhat.com>
Documentation/filesystems/overlayfs.txt
fs/overlayfs/Kconfig
fs/overlayfs/namei.c
fs/overlayfs/ovl_entry.h
fs/overlayfs/super.c