Commit 1ede9868 authored Oct 21, 2015 by Johan Hedberg Committed by Marcel Holtmann Oct 21, 2015

Bluetooth: Remove redundant (and possibly wrong) flag clearing



There's no need to clear the HCI_CONN_ENCRYPT_PEND flag in
smp_failure. In fact, this may cause the encryption tracking to get
out of sync as this has nothing to do with HCI activity.

Signed-off-by: Johan Hedberg <johan.hedberg@intel.com>
Signed-off-by: Marcel Holtmann <marcel@holtmann.org>

parent b5c2b621

net/bluetooth/smp.c

+0 −1

Original line number	Diff line number	Diff line
		@@ -811,7 +811,6 @@ static void smp_failure(struct l2cap_conn *conn, u8 reason)
		smp_send_cmd(conn, SMP_CMD_PAIRING_FAIL, sizeof(reason),
		&reason);

		clear_bit(HCI_CONN_ENCRYPT_PEND, &hcon->flags);
		mgmt_auth_failed(hcon, HCI_ERROR_AUTH_FAILURE);

		if (chan->data)