On Tue, 13 Sep 2022 14:04:38 +0200
Janusz Krzysztofik <[email protected]> wrote:

> Since we pass NULL to igt_audio_driver_unload(), realloc() it calls
> allocates new memory for us.  Free it each time we no longer need it,
> before dropping a pointer to it, to avoid memory leaking.
> 
> Signed-off-by: Janusz Krzysztofik <[email protected]>

Reviewed-by: Mauro Carvalho Chehab <[email protected]>

> ---
>  tests/core_hotunplug.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/tests/core_hotunplug.c b/tests/core_hotunplug.c
> index a0d29b799a..733ea8efb0 100644
> --- a/tests/core_hotunplug.c
> +++ b/tests/core_hotunplug.c
> @@ -187,6 +187,7 @@ static void driver_bind(struct hotunplug *priv, int 
> timeout)
>       if (priv->snd_driver) {
>               igt_info("Realoading %s\n", priv->snd_driver);
>               igt_kmod_load(priv->snd_driver, NULL);
> +             free(priv->snd_driver);
>               priv->snd_driver = NULL;
>       }
>  }

Reply via email to