summaryrefslogtreecommitdiffstats
path: root/crypto/twofish_common.c
diff options
context:
space:
mode:
authorLinus Torvalds2011-02-16 17:56:55 +0100
committerLinus Torvalds2011-02-16 17:56:55 +0100
commit3abb17e82f08628b59e20d8cbcb55e2204180f69 (patch)
tree890a411ebda28a9b9e6c90520429554a9e7fb95a /crypto/twofish_common.c
parentLinux 2.6.38-rc5 (diff)
downloadkernel-qcow2-linux-3abb17e82f08628b59e20d8cbcb55e2204180f69.tar.gz
kernel-qcow2-linux-3abb17e82f08628b59e20d8cbcb55e2204180f69.tar.xz
kernel-qcow2-linux-3abb17e82f08628b59e20d8cbcb55e2204180f69.zip
vfs: fix BUG_ON() in fs/namei.c:1461
When Al moved the nameidata_dentry_drop_rcu_maybe() call into the do_follow_link function in commit 844a391799c2 ("nothing in do_follow_link() is going to see RCU"), he mistakenly left the BUG_ON(inode != path->dentry->d_inode); behind. Which would otherwise be ok, but that BUG_ON() really needs to be _after_ dropping RCU, since the dentry isn't necessarily stable otherwise. So complete the code movement in that commit, and move the BUG_ON() into do_follow_link() too. This means that we need to pass in 'inode' as an argument (just for this one use), but that's a small thing. And eventually we may be confident enough in our path lookup that we can just remove the BUG_ON() and the unnecessary inode argument. Reported-and-tested-by: Eric Dumazet <eric.dumazet@gmail.com> Acked-by: Al Viro <viro@zeniv.linux.org.uk> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'crypto/twofish_common.c')
0 files changed, 0 insertions, 0 deletions