Return-Path: From: To: CC: , , Date: Mon, 31 Oct 2011 18:00:20 +0200 Subject: RE: [PATCH] Bluetooth: Revert: Fix L2CAP connection establishment Message-ID: References: <1319621002-7122-1-git-send-email-arkadiusz.lichwa@tieto.com> <1319621002-7122-2-git-send-email-arkadiusz.lichwa@tieto.com> In-Reply-To: <1319621002-7122-2-git-send-email-arkadiusz.lichwa@tieto.com> Content-Type: text/plain; charset="iso-8859-2" MIME-Version: 1.0 Sender: linux-bluetooth-owner@vger.kernel.org List-ID: Hi >Subject: [PATCH] Bluetooth: Revert: Fix L2CAP connection establishment > >This reverts commit 330605423ca6eafafb8dcc27502bce1c585d1b06. >The commit introduces regression when two 2.1 devices attempt >establish rfcomm channel. Such connection is refused since there's >a security block issue on l2cap. It means the link is unencrypted. > >2011-09-16 18:08:46.567616 < ACL data: handle 1 flags 0x00 dlen 24 > 0000: 14 00 40 00 06 00 02 00 0f 35 03 19 12 00 ff ff >..@......5....?? > 0010: 35 05 0a 00 00 ff ff 00 5....??. >2011-09-16 18:08:46.572377 > HCI Event: Number of Completed Packets >(0x13) plen 5 > handle 1 packets 1 >2011-09-16 18:08:46.577931 > ACL data: handle 1 flags 0x02 dlen 88 > L2CAP(d): cid 0x0040 len 84 [psm 0] > 0000: 07 00 02 00 4f 00 4c 35 4a 35 48 09 00 00 0a 00 >....O.L5J5H..... > 0010: 01 00 00 09 00 01 35 03 19 12 00 09 00 05 35 03 >......5.......5. > 0020: 19 10 02 09 00 09 35 08 35 06 19 12 00 09 01 02 >......5.5....... > 0030: 09 02 00 09 01 02 09 02 01 09 00 0a 09 02 02 09 >................ > 0040: 00 00 09 02 03 09 00 00 09 02 04 28 01 09 02 05 >...........(.... > 0050: 09 00 02 00 .... >2011-09-16 18:08:46.626057 < HCI Command: Authentication Requested >(0x01|0x0011) plen 2 > handle 1 >2011-09-16 18:08:46.627614 > HCI Event: Command Status (0x0f) plen 4 > Authentication Requested (0x01|0x0011) status 0x00 ncmd 1 >2011-09-16 18:08:46.627675 > HCI Event: Link Key Request (0x17) plen 6 > bdaddr 00:00:F2:6A:29:69 >2011-09-16 18:08:46.634999 < HCI Command: Link Key Request Reply >(0x01|0x000b) plen 22 > bdaddr 00:00:F2:6A:29:69 key 58CD393179FC902E5E8F512A855EE532 >2011-09-16 18:08:46.683278 > HCI Event: Command Complete (0x0e) plen 10 > Link Key Request Reply (0x01|0x000b) ncmd 1 > status 0x00 bdaddr 00:00:F2:6A:29:69 >2011-09-16 18:08:46.764729 > HCI Event: Auth Complete (0x06) plen 3 > status 0x00 handle 1 >2011-09-16 18:08:46.764821 < ACL data: handle 1 flags 0x00 dlen 12 > 0000: 08 00 01 00 02 05 04 00 03 00 41 00 ..........A. >2011-09-16 18:08:46.764851 > HCI Event: Command Status (0x0f) plen 4 > Unknown (0x00|0x0000) status 0x00 ncmd 2 >2011-09-16 18:08:46.768117 > HCI Event: Number of Completed Packets >(0x13) plen 5 > handle 1 packets 1 >2011-09-16 18:08:46.770894 > ACL data: handle 1 flags 0x02 dlen 16 > L2CAP(s): Connect rsp: dcid 0x0000 scid 0x0041 result 3 status 0 > Connection refused - security block >2011-09-16 18:08:49.000691 < ACL data: handle 1 flags 0x00 dlen 12 > 0000: 08 00 01 00 06 06 04 00 40 00 40 00 ........@.@. >2011-09-16 18:08:49.015675 > HCI Event: Number of Completed Packets >(0x13) plen 5 > handle 1 packets 1 >2011-09-16 18:08:49.016927 > ACL data: handle 1 flags 0x02 dlen 12 > L2CAP(s): Disconn rsp: dcid 0x0040 scid 0x0040 >2011-09-16 18:08:51.009480 < HCI Command: Disconnect (0x01|0x0006) plen >3 > handle 1 reason 0x13 > Reason: Remote User Terminated Connection >2011-09-16 18:08:51.011525 > HCI Event: Command Status (0x0f) plen 4 > Disconnect (0x01|0x0006) status 0x00 ncmd 1 >2011-09-16 18:08:51.123494 > HCI Event: Disconn Complete (0x05) plen 4 > status 0x00 handle 1 reason 0x16 > Reason: Connection Terminated by Local Host > >Signed-off-by: Arek Lichwa >--- > net/bluetooth/hci_conn.c | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > >diff --git a/net/bluetooth/hci_conn.c b/net/bluetooth/hci_conn.c >index c1c597e..e0af723 100644 >--- a/net/bluetooth/hci_conn.c >+++ b/net/bluetooth/hci_conn.c >@@ -673,7 +673,7 @@ int hci_conn_security(struct hci_conn *conn, __u8 >sec_level, __u8 auth_type) > goto encrypt; > > auth: >- if (test_bit(HCI_CONN_ENCRYPT_PEND, &conn->pend)) >+ if (test_and_set_bit(HCI_CONN_ENCRYPT_PEND, &conn->pend)) > return 0; > > if (!hci_conn_auth(conn, sec_level, auth_type)) >-- >1.7.6 Is it something unclear in my original post ? Any comments ? Regression here is quite obvious in my opinion. /Arek