linux-kernel - Re: [PATCH v4.4/bluetooth 1/2] Bluetooth: Consolidate encryption handling in hci_encrypt

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <0804be81-ebbf-8132-a619-c0c740a71f51@cisco.com>
Date:   Thu, 15 Oct 2020 11:18:39 +0000
From:   "Hans-Christian Egtvedt (hegtvedt)" <hegtvedt@...co.com>
To:     Greg KH <gregkh@...uxfoundation.org>
CC:     "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        "Luiz Augusto von Dentz" <luiz.von.dentz@...el.com>,
        Marcel Holtmann <marcel@...tmann.org>
Subject: Re: [PATCH v4.4/bluetooth 1/2] Bluetooth: Consolidate encryption
 handling in hci_encrypt_cfm

On 15/10/2020 11:57, Greg KH wrote:
> On Thu, Oct 15, 2020 at 09:43:32AM +0200, Hans-Christian Noren Egtvedt wrote:
>> From: Luiz Augusto von Dentz <luiz.von.dentz@...el.com>
>>
>> This makes hci_encrypt_cfm calls hci_connect_cfm in case the connection
>> state is BT_CONFIG so callers don't have to check the state.
>>
>> Signed-off-by: Luiz Augusto von Dentz <luiz.von.dentz@...el.com>
>> Signed-off-by: Marcel Holtmann <marcel@...tmann.org>
>> (cherry picked from commit 3ca44c16b0dcc764b641ee4ac226909f5c421aa3)
> 
> This is only in 5.8, what about all the other stable kernels?
> 
>> (cherry picked from commit 0a60996c7fa7010ea00d9b62fb6996d908a01ead)
> 
> Where is this commit from?  I don't see it in Linus's tree.

Ops, my bad, I have a Linux 4.9 branch, which also includes the Android
4.9 kernel changes from Google. And since I noticed the patches in my
(merged from Android latest) linux-4.9.y branch, I thought the patches
were applied through all stable releases 4.9 and up.

>> ---
>> AFAICT, fixing CVE 2020-10135 Bluetooth impersonation attacks have been
>> left out for the 4.4 stable kernel. I cherry picked what I assume are
>> the appropriate two patches missing from the 4.9 stable kernel. Please
>> add them to upcoming 4.4 stable releases.
> 
> Why are you merging 2 commits together?  Please provide backports for
> all stable kernels, if you want to see this in the 4.4.y tree.  We can
> not have someone move from an older tree to a newer one and have a
> regression.

Agreed, I have managed to trick myself into thinking the 4.4.y branch
was left out, but I assume these patches are required for all LTS branches.

That is a bigger job AFAIK, the newer branches might need additional
patches. Perhaps Luiz can assist?

-- 
Best regards, Hans-Christian Noren Egtvedt

Download attachment "pEpkey.asc" of type "application/pgp-keys" (1813 bytes)