aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJulien Isorce <julien.isorce@gmail.com>2017-03-10 17:16:05 +0000
committerEmil Velikov <emil.l.velikov@gmail.com>2017-04-12 13:30:21 +0100
commit99468c236f09211fcc4716efd400c039079c9a40 (patch)
treed76e880b7956afb7d5aea435856deda26c72c13a
parentc0a73ddda801c6cde0183735625dea95842f9c27 (diff)
downloadmesa3d-99468c236f09211fcc4716efd400c039079c9a40.tar.gz
winsys/radeon: check null in radeon_cs_create_fence
Fixes the following segmentation fault: radeon_drm_cs_add_buffer (bo=0x0) at radeon_drm_cs.c -> if (!bo->handle) (gdb) bt 0 radeon_drm_cs_add_buffer (bo=0x0) at radeon_drm_cs.c 1 0x00007fffe73575de in radeon_cs_create_fence radeon_drm_cs.c 2 0x00007fffe7358c48 in radeon_drm_cs_flush radeon_drm_cs.c Signed-off-by: Julien Isorce <jisorce@oblong.com> Signed-off-by: Marek Olšák <marek.olsak@amd.com> (cherry picked from commit d09edb01468ca385b6a8ffe29ac434dc42a78d07) Nominated-by: Emil Velikov <emil.velikov@collabora.com>
-rw-r--r--src/gallium/winsys/radeon/drm/radeon_drm_cs.c3
1 files changed, 3 insertions, 0 deletions
diff --git a/src/gallium/winsys/radeon/drm/radeon_drm_cs.c b/src/gallium/winsys/radeon/drm/radeon_drm_cs.c
index 8f9e8a60593..fd52a405c00 100644
--- a/src/gallium/winsys/radeon/drm/radeon_drm_cs.c
+++ b/src/gallium/winsys/radeon/drm/radeon_drm_cs.c
@@ -750,6 +750,9 @@ radeon_cs_create_fence(struct radeon_winsys_cs *rcs)
/* Create a fence, which is a dummy BO. */
fence = cs->ws->base.buffer_create(&cs->ws->base, 1, 1,
RADEON_DOMAIN_GTT, RADEON_FLAG_HANDLE);
+ if (!fence)
+ return NULL;
+
/* Add the fence as a dummy relocation. */
cs->ws->base.cs_add_buffer(rcs, fence,
RADEON_USAGE_READWRITE, RADEON_DOMAIN_GTT,