Re: [flashrom] [PATCH] CID1130012: Double free

[Stefan Tauner]

On Tue, 19 Nov 2013 20:35:57 +0100
Stefan Reinauer  wrote:
> CID1130012: Double free
>
> Depending on the implementation of the deallocator function, the memory may by
> placed on the free list more than once.
> In pony_spi_init: Memory is deallocated more than once
>
> Signed-off-by: Stefan Reinauer
> Index: pony_spi.c
>
> --- pony_spi.c	(revision 1763)
> +++ pony_spi.c	(working copy)
> @@ -140,6 +140,7 @@
>    } else if (arg && !strlen(arg)) {
>    	msg_perr("Error: Missing argument for programmer type.\n");
>    	free(arg);
> return 1;
> } else if (arg){
> msg_perr("Error: Invalid programmer type specified.\n");
> free(arg);

Acked-by: Stefan Tauner 
and committed in r1772, thanks!