Re: [PATCH] xl: properly dispose of vTPM struct instance

2025-01-16 Thread Anthony PERARD
On Tue, Jan 14, 2025 at 09:13:16AM +0100, Jan Beulich wrote: > The backend_domname field requires separate freeing; make sure to call > libxl_device_vtpm_dispose() also on respective error paths. > > Coverity-ID: 1638719 > Fixes: dde22055ac3a ("libxl: add vtpm support") > Signed-off-by: Jan Beulic

Re: [PATCH] xl: properly dispose of vTPM struct instance

2025-01-14 Thread Oleksii Kurochko
On 1/14/25 9:13 AM, Jan Beulich wrote: The backend_domname field requires separate freeing; make sure to call libxl_device_vtpm_dispose() also on respective error paths. Coverity-ID: 1638719 Fixes: dde22055ac3a ("libxl: add vtpm support") Signed-off-by: Jan Beulich R-Acked-By: Oleksii Kuroc

Re: [PATCH] xl: properly dispose of vTPM struct instance

2025-01-14 Thread Andrew Cooper
On 14/01/2025 8:13 am, Jan Beulich wrote: > The backend_domname field requires separate freeing; make sure to call > libxl_device_vtpm_dispose() also on respective error paths. > > Coverity-ID: 1638719 > Fixes: dde22055ac3a ("libxl: add vtpm support") > Signed-off-by: Jan Beulich Reviewed-by: And

[PATCH] xl: properly dispose of vTPM struct instance

2025-01-14 Thread Jan Beulich
The backend_domname field requires separate freeing; make sure to call libxl_device_vtpm_dispose() also on respective error paths. Coverity-ID: 1638719 Fixes: dde22055ac3a ("libxl: add vtpm support") Signed-off-by: Jan Beulich --- a/tools/xl/xl_vtpm.c +++ b/tools/xl/xl_vtpm.c @@ -44,12 +44,14 @@