On 12/23/10 09:01 AM, Trevor Woerner wrote:
> From: Trevor Woerner <[email protected]>
>
> Signed-off-by: Trevor Woerner <[email protected]>
> ---
> 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- [email protected]
Oracle Solaris Platform Engineering: X Window System
_______________________________________________
[email protected]: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel
