On Tue, 13 Sep 2022 14:04:38 +0200
Janusz Krzysztofik <janusz.krzysztofik@xxxxxxxxxxxxxxx> 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 <janusz.krzysztofik@xxxxxxxxxxxxxxx>
Reviewed-by: Mauro Carvalho Chehab <mchehab@xxxxxxxxxx>
> ---
> 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;
> }
> }
