[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