diff options
| author | Ingo Molnar <mingo@elte.hu> | 2006-07-01 13:35:46 +0200 |
|---|---|---|
| committer | Linus Torvalds <torvalds@g5.osdl.org> | 2006-07-01 18:55:57 +0200 |
| commit | ed6f7b10e657b98b4ba89385d02852c8bdf3980e (patch) | |
| tree | a53bfca204307322400c3fff1e75563233d38dfc /kernel/futex.c | |
| parent | [PATCH] pi-futex: fix mm_struct memory leak (diff) | |
| download | linux-ed6f7b10e657b98b4ba89385d02852c8bdf3980e.tar.xz linux-ed6f7b10e657b98b4ba89385d02852c8bdf3980e.zip | |
[PATCH] pi-futex: futex_wake() lockup fix
Fix futex_wake() exit condition bug when handling the robust-list with PI
futexes on them.
(reported by Ulrich Drepper, debugged by the lock validator.)
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Cc: Ulrich Drepper <drepper@redhat.com>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Diffstat (limited to 'kernel/futex.c')
| -rw-r--r-- | kernel/futex.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/kernel/futex.c b/kernel/futex.c index 22aa3c16ca73..15caf93e4a43 100644 --- a/kernel/futex.c +++ b/kernel/futex.c @@ -630,8 +630,10 @@ static int futex_wake(u32 __user *uaddr, int nr_wake) list_for_each_entry_safe(this, next, head, list) { if (match_futex (&this->key, &key)) { - if (this->pi_state) - return -EINVAL; + if (this->pi_state) { + ret = -EINVAL; + break; + } wake_futex(this); if (++ret >= nr_wake) break; |