gbm_dri: Fix DRIimage lookup callback
authorBenjamin Franzke <benjaminfranzke@googlemail.com>
Wed, 7 Mar 2012 16:21:20 +0000 (17:21 +0100)
committerBenjamin Franzke <benjaminfranzke@googlemail.com>
Wed, 7 Mar 2012 17:17:56 +0000 (18:17 +0100)
commitab6e081914ed2d9f8e166d6cfb9fd9998a45181d
treebc0cafe5e9490ca20a4681db15205ac8ba6ae741
parent6c1dea54457c1d19b67794f188c40c74c23ba762
gbm_dri: Fix DRIimage lookup callback

That is by making the dri extension variables static in gbm_dri.c.

The image_lookup_extension is provided by egl_dri2 when using x11 or wayland
platforms, when using the drm platform, gbm_dri has a wrapper for it.
Both use the same variables name image_lookup_extension.

Since -fvisibility=hidden was (probably by mistake) removed when converting to
automake, the "image_lookup_extension" symbol from egl_dri2.c became exported
in libEGL.so, so "image_lookup_extension" from gbm_dri.c was ignored.
This resulted in calling incorrect callbacks.

We cant make the image_lookup_extension static in egl_dri2.c right now,
since its used across multiple files.

Bugzilla: https://bugs.freedesktop.org/attachment.cgi?id=58099
Reviewed-by: Kristian Høgsberg <krh@bitplanet.net>
src/gbm/backends/dri/gbm_dri.c