Re: PATCH: Potential ressource leak in sys/dev/fb/vesa.c

From: Jung-uk Kim <jkim_at_FreeBSD.org>
Date: Tue, 6 Jul 2010 14:11:19 -0400
On Thursday 01 July 2010 02:52 pm, Yamagi Burmeister wrote:
> Hello,
> while tracking down a bug in vesa.c which caused a crash a friend
> of mine noticed a potential ressource leak in vesa.c. In line 841
> the execution is aborted via return (1); without freeing the
> already allocated resources. the attached patch changes the line to
> "goto fail;" which seems more correct.
>
> --- a/sys/dev/fb/vesa.c
> +++ b/sys/dev/fb/vesa.c
> _at__at_ -836,7 +836,7 _at__at_ vesa_bios_init(void)
>  		    "version 1.2 or later is required.\n",
>  		    ((vers & 0xf000) >> 12) * 10 + ((vers & 0x0f00) >> 8),
>  		    ((vers & 0x00f0) >> 4) * 10 + (vers & 0x000f));
> -		return (1);
>  		+		goto fail;
>  	}
>
>  	VESA_STRCPY(vesa_oemstr, buf->v_oemstr);

Fixed on HEAD.  Will be MFC'ed soon.

Thanks!

Jung-uk Kim
Received on Tue Jul 06 2010 - 16:11:35 UTC

This archive was generated by hypermail 2.4.0 : Wed May 19 2021 - 11:40:05 UTC