Return-Path: <linux-bluetooth-owner@vger.kernel.org>
MIME-Version: 1.0
In-Reply-To: <CABBYNZJ5cF0Gy-8UBiCaD5P1B4rqbS8NWOQ2YwfYjKYckQBL3Q@mail.gmail.com>
References: <20170809111859.10644-1-luiz.dentz@gmail.com> <521fe3ec-b1d8-1eda-183a-c2093568b7e3@cisco.com>
 <CABBYNZJ5cF0Gy-8UBiCaD5P1B4rqbS8NWOQ2YwfYjKYckQBL3Q@mail.gmail.com>
From: Luiz Augusto von Dentz <luiz.dentz@gmail.com>
Date: Thu, 10 Aug 2017 14:38:38 +0300
Message-ID: <CABBYNZKAu=j6aa4W6sHH3EVi4VswakrwFk+9kxkEyFy9cT3QLQ@mail.gmail.com>
Subject: Re: [PATCH BlueZ] device: Fix crash when connecting ATT with BR/EDR
 only device
To: =?UTF-8?Q?Per_Waag=C3=B8?= <pwaago@cisco.com>
Cc: "linux-bluetooth@vger.kernel.org" <linux-bluetooth@vger.kernel.org>
Content-Type: text/plain; charset="UTF-8"
Sender: linux-bluetooth-owner@vger.kernel.org
List-ID: <linux-bluetooth.vger.kernel.org>

Hi,

On Wed, Aug 9, 2017 at 3:59 PM, Luiz Augusto von Dentz
<luiz.dentz@gmail.com> wrote:
> Hi Per,
>
> On Wed, Aug 9, 2017 at 3:17 PM, Per Waag=C3=B8 <pwaago@cisco.com> wrote:
>> Hi Luiz,
>>
>>
>> On 09. aug. 2017 13:18, Luiz Augusto von Dentz wrote:
>>>
>>> From: Luiz Augusto von Dentz<luiz.von.dentz@intel.com>
>>>
>>> The fix introduced in 006213cf4d231ce66de273e96619474bd516359b only
>>> works for dual mode devices, if the device is BR/EDR the address type
>>> would match the type on browsing_req so it still possible to crash.
>>> ---
>>>   src/device.c | 38 +++++++++++++++++++++++---------------
>>>   1 file changed, 23 insertions(+), 15 deletions(-)
>>
>>
>> I have tested the patch. I can't reproduce the crash anymore when it's i=
n,
>> so this solves the problem. Thank you!
>
> Let check if it solves the crashes with other devices as well.

Applied.

--=20
Luiz Augusto von Dentz