diff options
| author | Adam Jackson <[email protected]> | 2019-10-02 16:26:48 -0400 |
|---|---|---|
| committer | Adam Jackson <[email protected]> | 2019-10-03 09:39:51 -0400 |
| commit | 1b87f4058de84d7a0bb4ead0c4f4b024d4cce8fb (patch) | |
| tree | ec1b272c1dcd4d471ae4c520685469b0fd3d2df8 /src/egl | |
| parent | c837872fbada93deb42f499de4325e0c41c9e3a1 (diff) | |
egl/dri2: Don't dlclose() the driver on dri2_load_driver_common failure
... because it's wrong to do so. The error path out of
dri2_initialize_drm ends with dri2_display_destroy, which calls
functions in the vtable we're trying to set up, so if we dlclose the
driver then those function pointers will point off into space and things
crash.
Noticed this because after !1923 eglinfo would crash when setting up the
GBM platform. This was something of a cascade failure, because my kernel
is too old for DRM_IOCTL_I915_GETPARAM to work without DRM_AUTH, so i965
wouldn't load. platform_drm.c then got very confused when it tries to
load swrast as a dri2 driver.
Reviewed-by: Eric Anholt <[email protected]>
Diffstat (limited to 'src/egl')
| -rw-r--r-- | src/egl/drivers/dri2/egl_dri2.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/src/egl/drivers/dri2/egl_dri2.c b/src/egl/drivers/dri2/egl_dri2.c index 82a0bed5c62..23d0e469427 100644 --- a/src/egl/drivers/dri2/egl_dri2.c +++ b/src/egl/drivers/dri2/egl_dri2.c @@ -633,10 +633,9 @@ dri2_load_driver_common(_EGLDisplay *disp, if (!extensions) return EGL_FALSE; - if (!dri2_bind_extensions(dri2_dpy, driver_extensions, extensions, false)) { - dlclose(dri2_dpy->driver); + if (!dri2_bind_extensions(dri2_dpy, driver_extensions, extensions, false)) return EGL_FALSE; - } + dri2_dpy->driver_extensions = extensions; dri2_bind_extensions(dri2_dpy, optional_driver_extensions, extensions, true); |