Commit 4d7936e8 authored by Zijun Hu's avatar Zijun Hu Committed by Luiz Augusto von Dentz
Browse files

Bluetooth: hci_sock: Reset cookie to zero in hci_sock_free_cookie() 



Reset cookie value to 0 instead of 0xffffffff in hci_sock_free_cookie()
since:
0         :  means cookie has not been assigned yet
0xffffffff:  means cookie assignment failure

Also fix generating cookie failure with usage shown below:
hci_sock_gen_cookie(sk)   // generate cookie
hci_sock_free_cookie(sk)  // free cookie
hci_sock_gen_cookie(sk)   // Can't generate cookie any more

Signed-off-by: default avatarZijun Hu <zijun.hu@oss.qualcomm.com>
Signed-off-by: default avatarLuiz Augusto von Dentz <luiz.von.dentz@intel.com>
parent b47c97f2

net/bluetooth/hci_sock.c

+1 −1
Original line number Diff line number Diff line
@@ -118,7 +118,7 @@ static void hci_sock_free_cookie(struct sock *sk) 
	int id = hci_pi(sk)->cookie;



	if (id) {

		hci_pi(sk)->cookie = 0xffffffff;

		hci_pi(sk)->cookie = 0;

		ida_free(&sock_cookie_ida, id);

	}

}