[PATCH i-g-t v2 1/9] tests/msm/msm_mapping: Fix memory leak

Kamil Konieczny kamil.konieczny at linux.intel.com
Fri Jul 4 15:02:48 UTC 2025


Hi Rob,
On 2025-07-02 at 12:02:41 -0700, Rob Clark wrote:
> We shouldn't leak the devcore.
> 
> Signed-off-by: Rob Clark <rob.clark at oss.qualcomm.com>

For whole series this is
Acked-by: Kamil Konieczny <kamil.konieczny at linux.intel.com>

> ---
>  tests/msm/msm_mapping.c | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/tests/msm/msm_mapping.c b/tests/msm/msm_mapping.c
> index ac20ca91d613..1749a670e70d 100644
> --- a/tests/msm/msm_mapping.c
> +++ b/tests/msm/msm_mapping.c
> @@ -215,6 +215,8 @@ do_mapping_test(struct msm_pipe *pipe, const char *buffername, bool write)
>  	ret = sscanf(s, "  - iova=%"PRIx64, &fault_addr);
>  	igt_fail_on(ret != 1);
>  	igt_fail_on(addr != fault_addr);
> +
> +	free(devcore);
>  }
>  
>  /*
> -- 
> 2.50.0
> 


More information about the igt-dev mailing list