[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v3] hw/cxl: Fix CFMW config memory leak
From: |
Jonathan Cameron |
Subject: |
Re: [PATCH v3] hw/cxl: Fix CFMW config memory leak |
Date: |
Wed, 31 May 2023 12:08:23 +0100 |
On Wed, 31 May 2023 09:51:43 +0200
Philippe Mathieu-Daudé <philmd@linaro.org> wrote:
> On 31/5/23 08:07, Li Zhijian wrote:
> > Allocate targets and targets[n] resources when all sanity checks are
> > passed to avoid memory leaks.
> >
> > Suggested-by: Philippe Mathieu-Daudé <philmd@linaro.org>
> > Signed-off-by: Li Zhijian <lizhijian@cn.fujitsu.com>
> > ---
> > V3: allocte further resource when we can't fail # Philippe
>
> Thanks for the v3!
>
> Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Thanks. I've added this near the top of my queue so will send
it out along with other similar fixes as a series for Michael
to consider picking up.
Jonathan
>
> > V2: Delete unnecesarry check
> > ---
> > hw/cxl/cxl-host.c | 12 ++++++------
> > 1 file changed, 6 insertions(+), 6 deletions(-)
>