summaryrefslogtreecommitdiffstats
path: root/fs/autofs4/root.c
diff options
context:
space:
mode:
authorIan Kent <raven@themaw.net>2012-10-11 08:00:33 +0800
committerBen Hutchings <ben@decadent.org.uk>2012-10-17 03:49:49 +0100
commitb7a9b74916d8e7ef5ac4970a71cdf411ae1286c8 (patch)
tree7121198fd654b747081ad29b98b2203ed9a1f91b /fs/autofs4/root.c
parente69e19924bdbfb7a44ac64c206d5cbcae57910ed (diff)
downloadlinux-stable-b7a9b74916d8e7ef5ac4970a71cdf411ae1286c8.tar.gz
linux-stable-b7a9b74916d8e7ef5ac4970a71cdf411ae1286c8.tar.bz2
linux-stable-b7a9b74916d8e7ef5ac4970a71cdf411ae1286c8.zip
autofs4 - fix reset pending flag on mount fail
commit 49999ab27eab6289a8e4f450e148bdab521361b2 upstream. In autofs4_d_automount(), if a mount fail occurs the AUTOFS_INF_PENDING mount pending flag is not cleared. One effect of this is when using the "browse" option, directory entry attributes show up with all "?"s due to the incorrect callback and subsequent failure return (when in fact no callback should be made). Signed-off-by: Ian Kent <ikent@redhat.com> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org> Signed-off-by: Ben Hutchings <ben@decadent.org.uk>
Diffstat (limited to 'fs/autofs4/root.c')
-rw-r--r--fs/autofs4/root.c6
1 files changed, 4 insertions, 2 deletions
diff --git a/fs/autofs4/root.c b/fs/autofs4/root.c
index f55ae23b137e..790fa6387644 100644
--- a/fs/autofs4/root.c
+++ b/fs/autofs4/root.c
@@ -392,10 +392,12 @@ static struct vfsmount *autofs4_d_automount(struct path *path)
ino->flags |= AUTOFS_INF_PENDING;
spin_unlock(&sbi->fs_lock);
status = autofs4_mount_wait(dentry);
- if (status)
- return ERR_PTR(status);
spin_lock(&sbi->fs_lock);
ino->flags &= ~AUTOFS_INF_PENDING;
+ if (status) {
+ spin_unlock(&sbi->fs_lock);
+ return ERR_PTR(status);
+ }
}
done:
if (!(ino->flags & AUTOFS_INF_EXPIRING)) {