diff options
| author | Peter Hurley <peter@hurleysoftware.com> | 2011-07-24 00:11:01 -0400 |
|---|---|---|
| committer | Jonas ABERG <jonas.aberg@stericsson.com> | 2011-10-28 11:01:20 +0200 |
| commit | e20918b6f1d0737a5d6d7ff55eb7378e1a48030d (patch) | |
| tree | e02fb1a7f6e246f5632b4792a08cba078258ac78 | |
| parent | fd78deefd3fb74d4f87e17f3e1b93692dab9aa25 (diff) | |
Bluetooth: sco: Fix lost wakeups waiting to accept socket
Fix race conditions which can cause lost wakeups (or missed signals)
while waiting to accept a sco socket connection.
Signed-off-by: Peter Hurley <peter@hurleysoftware.com>
Signed-off-by: Gustavo F. Padovan <padovan@profusion.mobi>
Change-Id: I905dbb0c2ab840960fc420b30fb5a8032afb8c9e
Reviewed-on: http://gerrit.lud.stericsson.com/gerrit/35567
Tested-by: Per VAHLNE <per.xx.vahlne@stericsson.com>
Reviewed-by: Jonas ABERG <jonas.aberg@stericsson.com>
| -rw-r--r-- | net/bluetooth/sco.c | 28 |
1 files changed, 14 insertions, 14 deletions
diff --git a/net/bluetooth/sco.c b/net/bluetooth/sco.c index 3b52e4729be..6e3c756a44e 100644 --- a/net/bluetooth/sco.c +++ b/net/bluetooth/sco.c @@ -592,30 +592,26 @@ static int sco_sock_accept(struct socket *sock, struct socket *newsock, int flag lock_sock(sk); - if (sk->sk_state != BT_LISTEN) { - err = -EBADFD; - goto done; - } - timeo = sock_rcvtimeo(sk, flags & O_NONBLOCK); BT_DBG("sk %p timeo %ld", sk, timeo); /* Wait for an incoming connection. (wake-one). */ add_wait_queue_exclusive(sk_sleep(sk), &wait); - while (!(ch = bt_accept_dequeue(sk, newsock))) { + while (1) { set_current_state(TASK_INTERRUPTIBLE); - if (!timeo) { - err = -EAGAIN; + + if (sk->sk_state != BT_LISTEN) { + err = -EBADFD; break; } - release_sock(sk); - timeo = schedule_timeout(timeo); - lock_sock(sk); + ch = bt_accept_dequeue(sk, newsock); + if (ch) + break; - if (sk->sk_state != BT_LISTEN) { - err = -EBADFD; + if (!timeo) { + err = -EAGAIN; break; } @@ -623,8 +619,12 @@ static int sco_sock_accept(struct socket *sock, struct socket *newsock, int flag err = sock_intr_errno(timeo); break; } + + release_sock(sk); + timeo = schedule_timeout(timeo); + lock_sock(sk); } - set_current_state(TASK_RUNNING); + __set_current_state(TASK_RUNNING); remove_wait_queue(sk_sleep(sk), &wait); if (err) |