[Bluetooth] Fix HID disconnect NULL pointer dereference
authorMarcel Holtmann <marcel@holtmann.org>
Fri, 20 Oct 2006 06:55:48 +0000 (08:55 +0200)
committerDavid S. Miller <davem@sunset.davemloft.net>
Fri, 20 Oct 2006 08:15:05 +0000 (01:15 -0700)
The latest HID disconnect sequence change introduced a NULL pointer
dereference. For the quirk to handle buggy remote HID implementations,
it is enough to wait for a potential control channel disconnect from
the remote side and it is also enough to wait only 500 msecs.

Signed-off-by: Marcel Holtmann <marcel@holtmann.org>
net/bluetooth/hidp/core.c

index 9a562cf7406bc89ba0ea125eec4979a2783744ce..66782010f82cbacf442eba7a2d6c64f17d8c0964 100644 (file)
@@ -507,12 +507,10 @@ static int hidp_session(void *arg)
 
        hidp_del_timer(session);
 
-       if (intr_sk->sk_state != BT_CONNECTED)
-               wait_event_timeout(*(ctrl_sk->sk_sleep), (ctrl_sk->sk_state == BT_CLOSED), HZ);
-
        fput(session->intr_sock->file);
 
-       wait_event_timeout(*(intr_sk->sk_sleep), (intr_sk->sk_state == BT_CLOSED), HZ);
+       wait_event_timeout(*(ctrl_sk->sk_sleep),
+               (ctrl_sk->sk_state == BT_CLOSED), msecs_to_jiffies(500));
 
        fput(session->ctrl_sock->file);