* [PATCH] Bluetooth: Fix minimum encryption key size check
@ 2019-07-01 11:13 Matias Karhumaa
2019-07-02 14:35 ` Marcel Holtmann
0 siblings, 1 reply; 2+ messages in thread
From: Matias Karhumaa @ 2019-07-01 11:13 UTC (permalink / raw)
To: Marcel Holtmann, Johan Hedberg, linux-bluetooth
Fixes minimum encryption key size check so that HCI_MIN_ENC_KEY_SIZE
is also allowed as stated in comment.
This bug caused connection problems with devices having min
encryption key size of 7.
This patch has been prepared against Linus' tree because previous
regression fix that introduced this bug is not in Bluetooth tree yet.
Fixes: 693cd8ce3f88 ("Bluetooth: Fix regression with minimum encryption key size alignment")
Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=203997
Signed-off-by: Matias Karhumaa <matias.karhumaa@gmail.com>
Cc: stable@vger.kernel.org
---
net/bluetooth/l2cap_core.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/net/bluetooth/l2cap_core.c b/net/bluetooth/l2cap_core.c
index 9f77432dbe38..5406d7cd46ad 100644
--- a/net/bluetooth/l2cap_core.c
+++ b/net/bluetooth/l2cap_core.c
@@ -1353,7 +1353,7 @@ static bool l2cap_check_enc_key_size(struct hci_conn *hcon)
* actually encrypted before enforcing a key size.
*/
return (!test_bit(HCI_CONN_ENCRYPT, &hcon->flags) ||
- hcon->enc_key_size > HCI_MIN_ENC_KEY_SIZE);
+ hcon->enc_key_size >= HCI_MIN_ENC_KEY_SIZE);
}
static void l2cap_do_start(struct l2cap_chan *chan)
--
2.17.1
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] Bluetooth: Fix minimum encryption key size check
2019-07-01 11:13 [PATCH] Bluetooth: Fix minimum encryption key size check Matias Karhumaa
@ 2019-07-02 14:35 ` Marcel Holtmann
0 siblings, 0 replies; 2+ messages in thread
From: Marcel Holtmann @ 2019-07-02 14:35 UTC (permalink / raw)
To: Matias Karhumaa; +Cc: Johan Hedberg, linux-bluetooth
Hi Matias,
> Fixes minimum encryption key size check so that HCI_MIN_ENC_KEY_SIZE
> is also allowed as stated in comment.
>
> This bug caused connection problems with devices having min
> encryption key size of 7.
>
> This patch has been prepared against Linus' tree because previous
> regression fix that introduced this bug is not in Bluetooth tree yet.
>
> Fixes: 693cd8ce3f88 ("Bluetooth: Fix regression with minimum encryption key size alignment")
> Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=203997
> Signed-off-by: Matias Karhumaa <matias.karhumaa@gmail.com>
> Cc: stable@vger.kernel.org
> ---
> net/bluetooth/l2cap_core.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
patch has been forwarded to Linus.
Regards
Marcel
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2019-07-02 14:35 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-07-01 11:13 [PATCH] Bluetooth: Fix minimum encryption key size check Matias Karhumaa
2019-07-02 14:35 ` Marcel Holtmann
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).