Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit b9cc553f authored by Gustavo Padovan's avatar Gustavo Padovan
Browse files

Bluetooth: hci_conn_auto_accept() doesn't need locking 



It doesn't really touch any sensitive information about hdev. So no need
to lock here.

Acked-by: default avatarMarcel Holtmann <marcel@holtmann.org>
Signed-off-by: default avatarGustavo F. Padovan <padovan@profusion.mobi>
parent db323f2f

net/bluetooth/hci_conn.c

+0 −4
Original line number Original line Diff line number Diff line
@@ -362,12 +362,8 @@ static void hci_conn_auto_accept(unsigned long arg) 
	struct hci_conn *conn = (void *) arg;

	struct hci_conn *conn = (void *) arg;

	struct hci_dev *hdev = conn->hdev;

	struct hci_dev *hdev = conn->hdev;





	hci_dev_lock(hdev);



	hci_send_cmd(hdev, HCI_OP_USER_CONFIRM_REPLY, sizeof(conn->dst),

	hci_send_cmd(hdev, HCI_OP_USER_CONFIRM_REPLY, sizeof(conn->dst),

								&conn->dst);

								&conn->dst);



	hci_dev_unlock(hdev);

}

}





struct hci_conn *hci_conn_add(struct hci_dev *hdev, int type, bdaddr_t *dst)

struct hci_conn *hci_conn_add(struct hci_dev *hdev, int type, bdaddr_t *dst)