]> git.proxmox.com Git - mirror_ubuntu-kernels.git/commit
vfs: fix d_ancestor() case in d_materialize_unique
authorMichel Lespinasse <walken@google.com>
Tue, 27 Mar 2012 00:32:44 +0000 (17:32 -0700)
committerLinus Torvalds <torvalds@linux-foundation.org>
Wed, 28 Mar 2012 16:54:34 +0000 (09:54 -0700)
commitb18dafc86bb879d2f38a1743985d7ceb283c2f4d
tree447070f77406615aebadaef03018d634c0da922c
parent6658a6991cef75719a21441aa0b7f8d6821534ee
vfs: fix d_ancestor() case in d_materialize_unique

In d_materialise_unique() there are 3 subcases to the 'aliased dentry'
case; in two subcases the inode i_lock is properly released but this
does not occur in the -ELOOP subcase.

This seems to have been introduced by commit 1836750115f2 ("fix loop
checks in d_materialise_unique()").

Signed-off-by: Michel Lespinasse <walken@google.com>
Cc: stable@vger.kernel.org # v3.0+
[ Added a comment, and moved the unlock to where we generate the -ELOOP,
  which seems to be more natural.

  You probably can't actually trigger this without a buggy network file
  server - d_materialize_unique() is for finding aliases on non-local
  filesystems, and the d_ancestor() case is for a hardlinked directory
  loop.

  But we should be robust in the case of such buggy servers anyway. ]
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
fs/dcache.c