xlib: do not cache return value of glXChooseVisual/glXGetVisualFromFBConfig
authorJohn Sheu <sheu@google.com>
Wed, 13 Apr 2016 20:57:42 +0000 (13:57 -0700)
committerAlejandro Piñeiro <apinheiro@igalia.com>
Fri, 15 Apr 2016 05:44:34 +0000 (07:44 +0200)
commitfe9d8cd79e9380e29eb92f97903e8cb79d25371a
tree79f9ccb4959ca34babf033bf63d2b1d968bee65b
parenteeff13315858fcb09eefba9a94e6bae5820572e0
xlib: do not cache return value of glXChooseVisual/glXGetVisualFromFBConfig

The returned XVisualInfo from glXChooseVisual/glXGetVisualFromFBConfig
is being cached in XMesaVisual.vishandle (and unconditionally
overwritten on subsequent calls).  However, these entry points are
specified to return XVisualInfo instances to be owned by the caller and
freed with XFree(), so the return values should not be retained.

With this change, XMesaVisual.vishandle is essentially unused and will
be removed in a subsequent change.

v2: update commit message

Reviewed-by: Alejandro Piñeiro <apinheiro@igalia.com>
src/mesa/drivers/x11/fakeglx.c