On Tue, 19 Oct 2010 11:59:23 -0400, Adam Jackson <[email protected]> wrote:
> render2.c: In function ‘__glXDisp_Map2d’:
> render2.c:127: warning: ‘u1’ may be used uninitialized in this function
> render2.c: In function ‘__glXDisp_Map1d’:
> render2.c:90: warning: ‘u1’ may be used uninitialized in this function

> -#define __GLX_MEM_COPY(dst,src,n)    if (src != NULL && dst != NULL) 
> memcpy(dst,src,n)
> +#define __GLX_MEM_COPY(dst,src,n)    memcpy(dst,src,n)

Looks like this actually needs to be a call to memmove instead -- every
use of this macro appears to have overlapped arguments. Seems like just
directly calling memmove would be cleaner anyway.

-- 
[email protected]

Attachment: pgpswhN0saT3M.pgp
Description: PGP signature

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