On Thu, Nov 28, 2024 at 10:24 AM Anastasia Belova <abelova@xxxxxxxxxxxxx> wrote:
Just a friendly reminder.
Merged, thanks.
Sorry for the delay.
Frediano
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@xxxxxxxxxxxxx>
> ---
> 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;
> }
> }