From c6bb41c28b2da269d5b692d1d0f4acbf3cdd4b5c Mon Sep 17 00:00:00 2001 From: Francisco Jerez Date: Thu, 28 Jun 2012 01:16:23 +0200 Subject: [PATCH] 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 --- src/gallium/state_trackers/clover/core/base.hpp | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) 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 - ref_ptr(const ref_ptr &ref) : p(NULL) { + ref_ptr(const ref_ptr &ref) : p(NULL) { reset(ref.p); } -- 2.30.2