Bluetooth: Stop sabotaging list poisoning
authorAlexey Dobriyan <adobriyan@gmail.com>
Wed, 10 Jun 2015 17:28:33 +0000 (20:28 +0300)
committerMarcel Holtmann <marcel@holtmann.org>
Wed, 10 Jun 2015 23:22:54 +0000 (01:22 +0200)
list_del() poisons pointers with special values, no need to overwrite them.

Signed-off-by: Alexey Dobriyan <adobriyan@gmail.com>
Signed-off-by: Marcel Holtmann <marcel@holtmann.org>
net/bluetooth/l2cap_core.c

index 07bd316d02baf7c429c56b6ac8f2f0288d60400a..51594fb7b9e72f80c06af1f04e9fd89b01aba9ba 100644 (file)
@@ -1601,7 +1601,7 @@ int l2cap_register_user(struct l2cap_conn *conn, struct l2cap_user *user)
 
        hci_dev_lock(hdev);
 
-       if (user->list.next || user->list.prev) {
+       if (!list_empty(&user->list)) {
                ret = -EINVAL;
                goto out_unlock;
        }
@@ -1631,12 +1631,10 @@ void l2cap_unregister_user(struct l2cap_conn *conn, struct l2cap_user *user)
 
        hci_dev_lock(hdev);
 
-       if (!user->list.next || !user->list.prev)
+       if (list_empty(&user->list))
                goto out_unlock;
 
        list_del(&user->list);
-       user->list.next = NULL;
-       user->list.prev = NULL;
        user->remove(conn, user);
 
 out_unlock:
@@ -1651,8 +1649,6 @@ static void l2cap_unregister_all_users(struct l2cap_conn *conn)
        while (!list_empty(&conn->users)) {
                user = list_first_entry(&conn->users, struct l2cap_user, list);
                list_del(&user->list);
-               user->list.next = NULL;
-               user->list.prev = NULL;
                user->remove(conn, user);
        }
 }