Tomas Carnecky wrote:
>      /* Copy the string so it can be changed */
>  
> -    tmpptr = mybuf = (char *) xalloc (strlen (address) + 1);
> +    tmpptr = mybuf = malloc (strlen (address) + 1);
>      strcpy (mybuf, address);

While I'd prefer to see those eventually change to strdup calls,
that's not part of this patch, and this patch seems complete as is.

Reviewed-by: Alan Coopersmith <[email protected]>

-- 
        -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

Reply via email to