diff options
author | Ulisses Furquim <ulissesf@gmail.com> | 2008-02-08 07:18:16 -0500 |
---|---|---|
committer | Linus Torvalds <torvalds@woody.linux-foundation.org> | 2008-02-08 12:22:22 -0500 |
commit | ac74c00e499ed276a965e5b5600667d5dc04a84a (patch) | |
tree | 81a34f797445a15b322ab48f9f69c272ef1e242d | |
parent | a4ffc0a0b240a29cbe489f6db9dae112a49ef1c1 (diff) |
inotify: fix check for one-shot watches before destroying them
As the IN_ONESHOT bit is never set when an event is sent we must check it
in the watch's mask and not in the event's mask.
Signed-off-by: Ulisses Furquim <ulissesf@gmail.com>
Reported-by: "Clem Taylor" <clem.taylor@gmail.com>
Tested-by: "Clem Taylor" <clem.taylor@gmail.com>
Cc: Amy Griffis <amy.griffis@hp.com>
Cc: Robert Love <rlove@google.com>
Cc: John McCutchan <ttb@tentacle.dhs.org>
Cc: <stable@kernel.org>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
-rw-r--r-- | fs/inotify_user.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/inotify_user.c b/fs/inotify_user.c index a336c9709f3c..3ab09a65c456 100644 --- a/fs/inotify_user.c +++ b/fs/inotify_user.c | |||
@@ -283,7 +283,7 @@ static void inotify_dev_queue_event(struct inotify_watch *w, u32 wd, u32 mask, | |||
283 | /* we can safely put the watch as we don't reference it while | 283 | /* we can safely put the watch as we don't reference it while |
284 | * generating the event | 284 | * generating the event |
285 | */ | 285 | */ |
286 | if (mask & IN_IGNORED || mask & IN_ONESHOT) | 286 | if (mask & IN_IGNORED || w->mask & IN_ONESHOT) |
287 | put_inotify_watch(w); /* final put */ | 287 | put_inotify_watch(w); /* final put */ |
288 | 288 | ||
289 | /* coalescing: drop this event if it is a dupe of the previous */ | 289 | /* coalescing: drop this event if it is a dupe of the previous */ |