On 03/13/14 11:40 AM, Eric Anholt wrote:
Keith Packard <kei...@keithp.com> writes:

Alan Coopersmith <alan.coopersm...@oracle.com> writes:

Flagged by cppcheck 1.64:
[glamor/glamor_core.c:536] -> [glamor/glamor_core.c:540]:
     (warning) Possible null pointer dereference: extension - otherwise
               it is redundant to check it against null.
[glamor/glamor_egl.c:620] -> [glamor/glamor_egl.c:622]:
     (warning) Possible null pointer dereference: extension - otherwise
               it is redundant to check it against null.

Signed-off-by: Alan Coopersmith <alan.coopersm...@oracle.com>
Reviewed-by: Eric Anholt <e...@anholt.net>

Alan -- I can merge these directly, but I think I'd rather have them
come through Eric's Glamor tree at this point so that he can sequence
things as he prefers.

OK, sounds good.  I've pulled 2/2 in, since 1/2's code is all deleted in
the giant new series.

Works for me - thanks.

--
        -Alan Coopersmith-              alan.coopersm...@oracle.com
         Oracle Solaris Engineering - http://blogs.oracle.com/alanc
_______________________________________________
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

Reply via email to