Mesa (master): egl/dri2: Check return value of __DRI2fence::create_fence()

Chad Versace chadversary at kemper.freedesktop.org
Wed May 6 15:10:14 UTC 2015


Module: Mesa
Branch: master
Commit: b0f410a2a05e046c31c32e169b9463cb6b4af8ae
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=b0f410a2a05e046c31c32e169b9463cb6b4af8ae

Author: Chad Versace <chad.versace at intel.com>
Date:   Tue May  5 19:05:20 2015 -0700

egl/dri2: Check return value of __DRI2fence::create_fence()

If it returns NULL, then return early with an error.

Reviewed-by: Marek Olšák <marek.olsak at amd.com>

---

 src/egl/drivers/dri2/egl_dri2.c |    7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/src/egl/drivers/dri2/egl_dri2.c b/src/egl/drivers/dri2/egl_dri2.c
index 1e843ed..8ed1dda 100644
--- a/src/egl/drivers/dri2/egl_dri2.c
+++ b/src/egl/drivers/dri2/egl_dri2.c
@@ -2232,6 +2232,13 @@ dri2_create_sync(_EGLDriver *drv, _EGLDisplay *dpy,
    switch (type) {
    case EGL_SYNC_FENCE_KHR:
       dri2_sync->fence = dri2_dpy->fence->create_fence(dri2_ctx->dri_context);
+      if (!dri2_sync->fence) {
+         /* Why did it fail? DRI doesn't return an error code, so we emit
+          * a generic EGL error that doesn't communicate user error. */
+         _eglError(EGL_BAD_ALLOC, "eglCreateSyncKHR");
+         free(dri2_sync);
+         return NULL;
+      }
       break;
 
    case EGL_SYNC_CL_EVENT_KHR:




More information about the mesa-commit mailing list