]> asedeno.scripts.mit.edu Git - linux.git/commit
configfs: fix a deadlock in configfs_symlink()
authorAl Viro <viro@zeniv.linux.org.uk>
Sat, 3 Aug 2019 15:51:18 +0000 (11:51 -0400)
committerChristoph Hellwig <hch@lst.de>
Wed, 11 Sep 2019 10:45:49 +0000 (12:45 +0200)
commit351e5d869e5ac10cb40c78b5f2d7dfc816ad4587
treebbff139362c2d49c1bfff6a843c7e32cfd32902b
parentf74c2bb98776e2de508f4d607cd519873065118e
configfs: fix a deadlock in configfs_symlink()

Configfs abuses symlink(2).  Unlike the normal filesystems, it
wants the target resolved at symlink(2) time, like link(2) would've
done.  The problem is that ->symlink() is called with the parent
directory locked exclusive, so resolving the target inside the
->symlink() is easily deadlocked.

Short of really ugly games in sys_symlink() itself, all we can
do is to unlock the parent before resolving the target and
relock it after.  However, that invalidates the checks done
by the caller of ->symlink(), so we have to
* check that dentry is still where it used to be
(it couldn't have been moved, but it could've been unhashed)
* recheck that it's still negative (somebody else
might've successfully created a symlink with the same name
while we were looking the target up)
* recheck the permissions on the parent directory.

Cc: stable@vger.kernel.org
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Signed-off-by: Christoph Hellwig <hch@lst.de>
fs/configfs/symlink.c