From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Date: Mon, 9 Jun 2014 16:33:02 +0300 From: Johan Hedberg To: Jiri Slaby Cc: stable@vger.kernel.org, linux-kernel@vger.kernel.org, Marcel Holtmann Subject: Re: [PATCH 3.12 064/146] Bluetooth: Fix redundant encryption request for reauthentication Message-ID: <20140609133302.GA10377@t440s.lan> References: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Sender: linux-kernel-owner@vger.kernel.org List-ID: Hi, On Mon, Jun 09, 2014, Jiri Slaby wrote: > 3.12-stable review patch. If anyone has any objections, please let me know. > > =============== > > commit 09da1f3463eb81d59685df723b1c5950b7570340 upstream. > > When we're performing reauthentication (in order to elevate the > security level from an unauthenticated key to an authenticated one) we > do not need to issue any encryption command once authentication > completes. Since the trigger for the encryption HCI command is the > ENCRYPT_PEND flag this flag should not be set in this scenario. > Instead, the REAUTH_PEND flag takes care of all necessary steps for > reauthentication. > > Signed-off-by: Johan Hedberg > Signed-off-by: Marcel Holtmann > Signed-off-by: Jiri Slaby > --- > net/bluetooth/hci_conn.c | 9 ++++++--- > 1 file changed, 6 insertions(+), 3 deletions(-) This one seems to be causing a regression, so it's probably best to keep it out of stable trees for now: https://bugzilla.kernel.org/show_bug.cgi?id=77541 Johan