Mesa (master): i965/sync: Don't crash when deleting sync object

Chad Versace chadversary at kemper.freedesktop.org
Thu May 7 15:13:12 UTC 2015


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

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

i965/sync: Don't crash when deleting sync object

Don't pass NULL to drm_intel_bo_unreference(). It doesn't like that.

Bug found by code inspection.

Reviewed-by: Daniel Stone <daniels at collabora.com>
Reviewed-by: Kenneth Graunke <kenneth at whitecape.org>

---

 src/mesa/drivers/dri/i965/intel_syncobj.c |    4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/src/mesa/drivers/dri/i965/intel_syncobj.c b/src/mesa/drivers/dri/i965/intel_syncobj.c
index e500fa0..3bc32df 100644
--- a/src/mesa/drivers/dri/i965/intel_syncobj.c
+++ b/src/mesa/drivers/dri/i965/intel_syncobj.c
@@ -59,7 +59,9 @@ intel_delete_sync_object(struct gl_context *ctx, struct gl_sync_object *s)
 {
    struct intel_sync_object *sync = (struct intel_sync_object *)s;
 
-   drm_intel_bo_unreference(sync->bo);
+   if (sync->bo)
+      drm_intel_bo_unreference(sync->bo);
+
    free(sync);
 }
 




More information about the mesa-commit mailing list