[PATCH] usbredirtestclient: fix memory leak

Anastasia Belova abelova at astralinux.ru
Thu Nov 28 10:18:06 UTC 2024


Just a friendly reminder.

12.11.2024 16:59, Anastasia Belova пишет:
> free dynamic memory pointed by data before returning
> from function.
>
> Found by Linux Verification Center (linuxtesting.org) with SVACE.
>
> Signed-off-by: Anastasia Belova <abelova at astralinux.ru>
> ---
>   usbredirtestclient/usbredirtestclient.c | 1 +
>   1 file changed, 1 insertion(+)
>
> diff --git a/usbredirtestclient/usbredirtestclient.c b/usbredirtestclient/usbredirtestclient.c
> index 62b72e8..6da87ec 100644
> --- a/usbredirtestclient/usbredirtestclient.c
> +++ b/usbredirtestclient/usbredirtestclient.c
> @@ -409,6 +409,7 @@ static int usbredirtestclient_cmdline_ctrl(void)
>               }
>               if (!arg || *endptr != '\0') {
>                   printf("Missing or invalid data byte(s)\n");
> +                free(data);
>                   return 0;
>               }
>           }


More information about the Spice-devel mailing list