From: Francisco Jerez <currojerez@riseup.net>
Date: Wed, 27 Jun 2012 23:16:23 +0000 (+0200)
Subject: clover: Define non-templated copy constructor for clover::ref_ptr.
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=c6bb41c28b2da269d5b692d1d0f4acbf3cdd4b5c;p=mesa.git

clover: Define non-templated copy constructor for clover::ref_ptr.

The templated copy constructor doesn't prevent the compiler from
emitting a default copy constructor, which leads to inconsistent
memory handling and was reported to cause segfaults when doing event
manipulation.

Reported-by: Tom Stellard <thomas.stellard@amd.com>
---

diff --git a/src/gallium/state_trackers/clover/core/base.hpp b/src/gallium/state_trackers/clover/core/base.hpp
index 19053f39235..2a2e26a141a 100644
--- a/src/gallium/state_trackers/clover/core/base.hpp
+++ b/src/gallium/state_trackers/clover/core/base.hpp
@@ -88,8 +88,7 @@ namespace clover {
          reset(q);
       }
 
-      template<typename S>
-      ref_ptr(const ref_ptr<S> &ref) : p(NULL) {
+      ref_ptr(const ref_ptr<T> &ref) : p(NULL) {
          reset(ref.p);
       }