Mesa (master): clover: Destory pipe_screen when device does not support compute v2

Tom Stellard tstellar at kemper.freedesktop.org
Thu May 8 23:40:58 UTC 2014


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

Author: Tom Stellard <thomas.stellard at amd.com>
Date:   Thu May  8 21:08:32 2014 -0400

clover: Destory pipe_screen when device does not support compute v2

v2:
  - Make sure screen was successfully created before destroying it.

Cc: "10.2" <mesa-stable at lists.freedesktop.org>
Reviewed-by: Francisco Jerez <currojerez at riseup.net>

---

 src/gallium/state_trackers/clover/core/device.cpp |    5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/src/gallium/state_trackers/clover/core/device.cpp b/src/gallium/state_trackers/clover/core/device.cpp
index 2f84677..bc3e3e6 100644
--- a/src/gallium/state_trackers/clover/core/device.cpp
+++ b/src/gallium/state_trackers/clover/core/device.cpp
@@ -42,8 +42,11 @@ namespace {
 device::device(clover::platform &platform, pipe_loader_device *ldev) :
    platform(platform), ldev(ldev) {
    pipe = pipe_loader_create_screen(ldev, PIPE_SEARCH_DIR);
-   if (!pipe || !pipe->get_param(pipe, PIPE_CAP_COMPUTE))
+   if (!pipe || !pipe->get_param(pipe, PIPE_CAP_COMPUTE)) {
+      if (pipe)
+         pipe->destroy(pipe);
       throw error(CL_INVALID_DEVICE);
+   }
 }
 
 device::~device() {




More information about the mesa-commit mailing list