ACK: [PATCH] utilities: kernelscan: fix memory leak

Alex Hung alex.hung at canonical.com
Fri May 10 01:08:09 UTC 2013


On 05/07/2013 08:20 AM, Colin King wrote:
> From: Colin Ian King <colin.king at canonical.com>
>
> Coverity CID #997339, Resource leak
>
> Fix memory leak, free line when we return.
>
> Signed-off-by: Colin Ian King <colin.king at canonical.com>
> ---
>   src/utilities/kernelscan.c | 1 +
>   1 file changed, 1 insertion(+)
>
> diff --git a/src/utilities/kernelscan.c b/src/utilities/kernelscan.c
> index 95dc8a2..9acaa4e 100644
> --- a/src/utilities/kernelscan.c
> +++ b/src/utilities/kernelscan.c
> @@ -777,6 +777,7 @@ static int parse_kernel_message(parser *p, token *t)
>   	for (;;) {
>   		ret = get_token(p, t);
>   		if (ret == EOF) {
> +			free(line);
>   			return EOF;
>   		}
>
>
Acked-by: Alex Hung <alex.hung at canonical.com>



More information about the fwts-devel mailing list