On 12/23/10 09:01 AM, Trevor Woerner wrote:
> From: Trevor Woerner <twoer...@gmail.com>
>
> Signed-off-by: Trevor Woerner <twoer...@gmail.com>
> ---
> bmtoa.c | 1 +
> 1 files changed, 1 insertions(+), 0 deletions(-)
>
> diff --git a/bmtoa.c b/bmtoa.c
> index bdd2078..8cd96b1 100644
> --- a/bmtoa.c
> +++ b/bmtoa.c
> @@ -208,6 +208,7 @@ print_scanline (unsigned int width,
> putchar ('\n');
> if (padded) dp++;
> }
> + free (scanline);
> return;
> }
>
Is there any reason to malloc scanline at all? It appears
entirely unused. (Looks like someone planned to put characters
into it, then print that string, but instead just calls putchar()
directly for each one.)
--
-Alan Coopersmith- alan.coopersm...@oracle.com
Oracle Solaris Platform Engineering: X Window System
_______________________________________________
xorg-devel@lists.x.org: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel
