Re: [usbredir v1] usbredirhost: fix leak on error

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hey

On Wed, 2016-08-03 at 18:11 +0200, Victor Toso wrote:
> Pointed by coverity:
>  17. usbredir-0.7.1/usbredirhost/usbredirhost.c:2306: leaked_storage:
>  Returning without freeing "data" leaks the storage that it points to.
>  #  2304|
>  #  usbredirhost_bulk_packet_complete(transfer->transfer);
>  #  2305|       }
>  #  2306|-> }
>  #  2307|
>  #  2308|   static void usbredirhost_iso_packet(void *priv, uint64_t
>  #  id,
> ---
>  usbredirhost/usbredirhost.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/usbredirhost/usbredirhost.c b/usbredirhost/usbredirhost.c
> index 3e80677..2b3ee74 100644
> --- a/usbredirhost/usbredirhost.c
> +++ b/usbredirhost/usbredirhost.c
> @@ -2280,6 +2280,7 @@ static void usbredirhost_bulk_packet(void *priv,
> uint64_t id,
>                                           transfer, BULK_TIMEOUT);
>  #else
>          r = LIBUSB_ERROR_INVALID_PARAM;
> +        free(buffer);
buffer is not declared in that function

Pavel
>          goto error;
>  #endif
>      } else {
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]