Re: [PATCH] hw/clx: Fix CFMW config memory leak

2023-05-30 Thread Zhijian Li (Fujitsu)
On 30/05/2023 23:14, Jonathan Cameron via wrote: > On Mon, 29 May 2023 15:59:56 +0800 > Li Zhijian wrote: > >> Only 'fw' pointer is marked as g_autofree, so we shoud free other >> resource manually in error path. >> > Good spot. > > Patch title typo > hw/cxl:> OMG, it was the 2nd time i have

Re: [PATCH] hw/clx: Fix CFMW config memory leak

2023-05-30 Thread Jonathan Cameron via
On Mon, 29 May 2023 15:59:56 +0800 Li Zhijian wrote: > Only 'fw' pointer is marked as g_autofree, so we shoud free other > resource manually in error path. > Good spot. Patch title typo hw/cxl: It's a bit annoying we can't handle this with more autofree magic. That would work for fw->targets

[PATCH] hw/clx: Fix CFMW config memory leak

2023-05-29 Thread Li Zhijian
Only 'fw' pointer is marked as g_autofree, so we shoud free other resource manually in error path. Signed-off-by: Li Zhijian --- hw/cxl/cxl-host.c | 10 -- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/hw/cxl/cxl-host.c b/hw/cxl/cxl-host.c index 034c7805b3e..dd1a7c83f71 1