[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <51c4dfe5daf8497bd99a013458d93b92@codeaurora.org>
Date: Tue, 09 Jun 2020 21:20:43 +0530
From: gubbaven@...eaurora.org
To: Markus Elfring <Markus.Elfring@....de>
Cc: linux-arm-msm@...r.kernel.org, linux-bluetooth@...r.kernel.org,
Balakrishna Godavarthi <bgodavar@...eaurora.org>,
hemantg@...eaurora.org, Johan Hedberg <johan.hedberg@...il.com>,
Marcel Holtmann <marcel@...tmann.org>,
Matthias Kaehlcke <mka@...omium.org>,
Rocky Liao <rjliao@...eaurora.org>,
Rob Herring <robh@...nel.org>,
Sean Paul <seanpaul@...omium.org>,
Claire Chang <tientzu@...omium.org>, yshavit@...gle.com,
linux-kernel@...r.kernel.org, kernel-janitors@...r.kernel.org
Subject: Re: [PATCH] Bluetooth: hci_qca: Fix double free during SSR timeout
Hi Markus,
Sure we will update in next patch set.
Regards,
Lakshmi Narayna.
On 2020-06-04 23:54, Markus Elfring wrote:
>> Due to race conditions between qca_hw_error and qca_controller_memdump
>> during SSR timeout,the same pointer is freed twice.
>
> This is an unfortunate software situation.
>
>
>> Which results to double free error.
>
> How do you think about to omit this sentence from the change
> description?
>
>
>> Now a lock is acquired while SSR state moved to timeout.
>
> I suggest to convert this information into an imperative wording.
>
> Would you like to add the tag “Fixes” to the commit message?
>
> Regards,
> Markus
Powered by blists - more mailing lists