diff options
author | Ian Kent <raven@themaw.net> | 2006-03-27 04:14:59 -0500 |
---|---|---|
committer | Linus Torvalds <torvalds@g5.osdl.org> | 2006-03-27 11:44:41 -0500 |
commit | 3e7b19198003fc25b11838e709f17d4fa173b2d7 (patch) | |
tree | 6d78904c52b5c96c1c6f59544658235ef2a3c1ab /fs | |
parent | 871f94344cea36b2ce91231f442f9f9298529712 (diff) |
[PATCH] autofs4: atomic var underflow
Fix accidental underflow of the atomic counter.
Signed-off-by: Ian Kent <raven@themaw.net>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Diffstat (limited to 'fs')
-rw-r--r-- | fs/autofs4/autofs_i.h | 2 | ||||
-rw-r--r-- | fs/autofs4/waitq.c | 6 |
2 files changed, 5 insertions, 3 deletions
diff --git a/fs/autofs4/autofs_i.h b/fs/autofs4/autofs_i.h index ff6239d57b4b..617fd7b37447 100644 --- a/fs/autofs4/autofs_i.h +++ b/fs/autofs4/autofs_i.h | |||
@@ -85,7 +85,7 @@ struct autofs_wait_queue { | |||
85 | pid_t tgid; | 85 | pid_t tgid; |
86 | /* This is for status reporting upon return */ | 86 | /* This is for status reporting upon return */ |
87 | int status; | 87 | int status; |
88 | atomic_t notified; | 88 | atomic_t notify; |
89 | atomic_t wait_ctr; | 89 | atomic_t wait_ctr; |
90 | }; | 90 | }; |
91 | 91 | ||
diff --git a/fs/autofs4/waitq.c b/fs/autofs4/waitq.c index 894d74671bd0..142ab6aa2aa1 100644 --- a/fs/autofs4/waitq.c +++ b/fs/autofs4/waitq.c | |||
@@ -263,7 +263,7 @@ int autofs4_wait(struct autofs_sb_info *sbi, struct dentry *dentry, | |||
263 | wq->tgid = current->tgid; | 263 | wq->tgid = current->tgid; |
264 | wq->status = -EINTR; /* Status return if interrupted */ | 264 | wq->status = -EINTR; /* Status return if interrupted */ |
265 | atomic_set(&wq->wait_ctr, 2); | 265 | atomic_set(&wq->wait_ctr, 2); |
266 | atomic_set(&wq->notified, 1); | 266 | atomic_set(&wq->notify, 1); |
267 | mutex_unlock(&sbi->wq_mutex); | 267 | mutex_unlock(&sbi->wq_mutex); |
268 | } else { | 268 | } else { |
269 | atomic_inc(&wq->wait_ctr); | 269 | atomic_inc(&wq->wait_ctr); |
@@ -273,9 +273,11 @@ int autofs4_wait(struct autofs_sb_info *sbi, struct dentry *dentry, | |||
273 | (unsigned long) wq->wait_queue_token, wq->len, wq->name, notify); | 273 | (unsigned long) wq->wait_queue_token, wq->len, wq->name, notify); |
274 | } | 274 | } |
275 | 275 | ||
276 | if (notify != NFY_NONE && atomic_dec_and_test(&wq->notified)) { | 276 | if (notify != NFY_NONE && atomic_read(&wq->notify)) { |
277 | int type; | 277 | int type; |
278 | 278 | ||
279 | atomic_dec(&wq->notify); | ||
280 | |||
279 | if (sbi->version < 5) { | 281 | if (sbi->version < 5) { |
280 | if (notify == NFY_MOUNT) | 282 | if (notify == NFY_MOUNT) |
281 | type = autofs_ptype_missing; | 283 | type = autofs_ptype_missing; |