Re: [PATCH v2 i-g-t 1/1] tests/intel/xe_eudebug_online: Fix run_online_client memory leak

["Grzegorzek, Dominik" <dominik.grzegorzek@intel.com>]

On Thu, 2024-12-05 at 11:26 +0100, Dominik Karol Piątkowski wrote:
> In run_online_client, buf is created by create_uc_buf, that calls
> intel_buf_create_full, that callocs and initializes it. This memory is
> never freed. Fix it by adding missing intel_buf_destroy.
> 
> Signed-off-by: Dominik Karol Piątkowski <dominik.karol.piatkowski@intel.com>
> ---
>  tests/intel/xe_eudebug_online.c | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/tests/intel/xe_eudebug_online.c b/tests/intel/xe_eudebug_online.c
> index 750350556..bf53b44ce 100644
> --- a/tests/intel/xe_eudebug_online.c
> +++ b/tests/intel/xe_eudebug_online.c
> @@ -1132,6 +1132,8 @@ static void run_online_client(struct xe_eudebug_client *c)
>  					   DRM_XE_DEBUG_METADATA_PROGRAM_MODULE,
>  					   2 * sizeof(*metadata));
>  
> +	intel_buf_destroy(buf);
> +

Acked-by: Dominik Grzegorzek <dominik.grzegorzek@intel.com>

Regards,
Dominik

>  	xe_eudebug_client_close_driver(c, fd);
>  }
>