Commit c14cc63a authored by Ian Kent's avatar Ian Kent Committed by Al Viro

autofs4 - fix get_next_positive_dentry()

The initialization condition in fs/autofs4/expire.c:get_next_positive_dentry()
appears to be incorrect. If prev == NULL I believe that root should be
returned.

Further down, at the current dentry check for it being simple_positive()
it looks like the d_lock for dentry p should be dropped instead of dentry
ret, otherwise when p is assinged to ret we end up with no lock on p and
a lost lock on ret, which leads to a deadlock.
Signed-off-by: default avatarIan Kent <raven@themaw.net>
Signed-off-by: default avatarAl Viro <viro@zeniv.linux.org.uk>
parent 6845a44a
...@@ -96,7 +96,7 @@ static struct dentry *get_next_positive_dentry(struct dentry *prev, ...@@ -96,7 +96,7 @@ static struct dentry *get_next_positive_dentry(struct dentry *prev,
struct dentry *p, *ret; struct dentry *p, *ret;
if (prev == NULL) if (prev == NULL)
return dget(prev); return dget(root);
spin_lock(&autofs4_lock); spin_lock(&autofs4_lock);
relock: relock:
...@@ -133,7 +133,7 @@ static struct dentry *get_next_positive_dentry(struct dentry *prev, ...@@ -133,7 +133,7 @@ static struct dentry *get_next_positive_dentry(struct dentry *prev,
spin_lock_nested(&ret->d_lock, DENTRY_D_LOCK_NESTED); spin_lock_nested(&ret->d_lock, DENTRY_D_LOCK_NESTED);
/* Negative dentry - try next */ /* Negative dentry - try next */
if (!simple_positive(ret)) { if (!simple_positive(ret)) {
spin_unlock(&ret->d_lock); spin_unlock(&p->d_lock);
p = ret; p = ret;
goto again; goto again;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment