summaryrefslogtreecommitdiffstats
path: root/net/bluetooth/hci_sock.c
diff options
context:
space:
mode:
authorMarcel Holtmann <marcel@holtmann.org>2016-08-30 05:00:34 +0200
committerMarcel Holtmann <marcel@holtmann.org>2016-09-19 20:19:34 +0200
commitdf1cb87af9f24527a8932e4d195d49ffab1168d2 (patch)
tree1128642458f20dfa9324f8539b7c275bdcf210b7 /net/bluetooth/hci_sock.c
parent3d4e2fb64111ffb5dc737daf25f5434bf39bee5f (diff)
downloadlinux-df1cb87af9f24527a8932e4d195d49ffab1168d2.tar.bz2
Bluetooth: Introduce helper functions for socket cookie handling
Instead of manually allocating cookie information each time, use helper functions for generating and releasing cookies. Signed-off-by: Marcel Holtmann <marcel@holtmann.org> Signed-off-by: Johan Hedberg <johan.hedberg@intel.com>
Diffstat (limited to 'net/bluetooth/hci_sock.c')
-rw-r--r--net/bluetooth/hci_sock.c41
1 files changed, 29 insertions, 12 deletions
diff --git a/net/bluetooth/hci_sock.c b/net/bluetooth/hci_sock.c
index d37c2243157b..804208d48368 100644
--- a/net/bluetooth/hci_sock.c
+++ b/net/bluetooth/hci_sock.c
@@ -84,6 +84,33 @@ u32 hci_sock_get_cookie(struct sock *sk)
return hci_pi(sk)->cookie;
}
+static bool hci_sock_gen_cookie(struct sock *sk)
+{
+ int id = hci_pi(sk)->cookie;
+
+ if (!id) {
+ id = ida_simple_get(&sock_cookie_ida, 1, 0, GFP_KERNEL);
+ if (id < 0)
+ id = 0xffffffff;
+
+ hci_pi(sk)->cookie = id;
+ get_task_comm(hci_pi(sk)->comm, current);
+ return true;
+ }
+
+ return false;
+}
+
+static void hci_sock_free_cookie(struct sock *sk)
+{
+ int id = hci_pi(sk)->cookie;
+
+ if (id) {
+ hci_pi(sk)->cookie = 0xffffffff;
+ ida_simple_remove(&sock_cookie_ida, id);
+ }
+}
+
static inline int hci_test_bit(int nr, const void *addr)
{
return *((const __u32 *) addr + (nr >> 5)) & ((__u32) 1 << (nr & 31));
@@ -753,7 +780,6 @@ static int hci_sock_release(struct socket *sock)
struct sock *sk = sock->sk;
struct hci_dev *hdev;
struct sk_buff *skb;
- int id;
BT_DBG("sock %p sk %p", sock, sk);
@@ -767,8 +793,6 @@ static int hci_sock_release(struct socket *sock)
atomic_dec(&monitor_promisc);
break;
case HCI_CHANNEL_CONTROL:
- id = hci_pi(sk)->cookie;
-
/* Send event to monitor */
skb = create_monitor_ctrl_close(sk);
if (skb) {
@@ -777,8 +801,7 @@ static int hci_sock_release(struct socket *sock)
kfree_skb(skb);
}
- hci_pi(sk)->cookie = 0xffffffff;
- ida_simple_remove(&sock_cookie_ida, id);
+ hci_sock_free_cookie(sk);
break;
}
@@ -1145,14 +1168,8 @@ static int hci_sock_bind(struct socket *sock, struct sockaddr *addr,
*/
if (haddr.hci_channel == HCI_CHANNEL_CONTROL) {
struct sk_buff *skb;
- int id;
-
- id = ida_simple_get(&sock_cookie_ida, 1, 0, GFP_KERNEL);
- if (id < 0)
- id = 0xffffffff;
- hci_pi(sk)->cookie = id;
- get_task_comm(hci_pi(sk)->comm, current);
+ hci_sock_gen_cookie(sk);
/* Send event to monitor */
skb = create_monitor_ctrl_open(sk);