Received-SPF: pass (google.com: domain of linux-bluetooth-owner@vger.kernel.org designates 23.128.96.33 as permitted sender) client-ip=23.128.96.33;
MIME-Version: 1.0
References: <20230918093517.2163899-1-kiran.k@intel.com>
In-Reply-To: <20230918093517.2163899-1-kiran.k@intel.com>
From:   Luiz Augusto von Dentz <luiz.dentz@gmail.com>
Date:   Mon, 18 Sep 2023 10:37:08 -0700
Message-ID: <CABBYNZKTQc-aQg+Yxzt7euZ1uDF0eQr_ipkFAOjuq+iPGKH3SQ@mail.gmail.com>
Subject: Re: [PATCH v1] Bluetooth: Release memory allocated to store supported codecs
To:     Kiran K <kiran.k@intel.com>
Cc:     linux-bluetooth@vger.kernel.org, ravishankar.srivatsa@intel.com,
        chethan.tumkur.narayan@intel.com
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
Precedence: bulk

Hi Kiran,

On Mon, Sep 18, 2023 at 5:04=E2=80=AFAM Kiran K <kiran.k@intel.com> wrote:
>
> Memory allocated to store local supported codecs needs to be freed
> when BT is powered off.
>
> Fixes: d0b137062b2d ("Bluetooth: hci_sync: Rework init stages")
> Signed-off-by: Kiran K <kiran.k@intel.com>
> ---
>  net/bluetooth/hci_sync.c | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/net/bluetooth/hci_sync.c b/net/bluetooth/hci_sync.c
> index 1be5763c261c..3640d73f9595 100644
> --- a/net/bluetooth/hci_sync.c
> +++ b/net/bluetooth/hci_sync.c
> @@ -5074,6 +5074,7 @@ int hci_dev_close_sync(struct hci_dev *hdev)
>         memset(hdev->eir, 0, sizeof(hdev->eir));
>         memset(hdev->dev_class, 0, sizeof(hdev->dev_class));
>         bacpy(&hdev->random_addr, BDADDR_ANY);
> +       hci_codec_list_clear(&hdev->local_codecs);
>
>         hci_dev_put(hdev);
>         return err;
> --
> 2.25.1

Ive sent a similar fix already:

https://patchwork.kernel.org/project/bluetooth/patch/20230915234742.3739283=
-2-luiz.dentz@gmail.com/


--=20
Luiz Augusto von Dentz