From d4984975ee0bdd68aeb9dfbc00258cce534b5a9b Mon Sep 17 00:00:00 2001 From: "kosak@google.com" Date: Sat, 14 Feb 2015 02:13:32 +0000 Subject: Add asserts to prevent mysterious hangs in a non-thread-safe gmock build. git-svn-id: http://googletest.googlecode.com/svn/trunk@706 861a406c-534a-0410-8894-cb66d6ee9925 --- include/gtest/internal/gtest-linked_ptr.h | 14 ++++++++++++-- 1 file changed, 12 insertions(+), 2 deletions(-) diff --git a/include/gtest/internal/gtest-linked_ptr.h b/include/gtest/internal/gtest-linked_ptr.h index b1362cd..3602942 100644 --- a/include/gtest/internal/gtest-linked_ptr.h +++ b/include/gtest/internal/gtest-linked_ptr.h @@ -110,7 +110,12 @@ class linked_ptr_internal { MutexLock lock(&g_linked_ptr_mutex); linked_ptr_internal const* p = ptr; - while (p->next_ != ptr) p = p->next_; + while (p->next_ != ptr) { + assert(p->next_ != this && + "Trying to join() a linked ring we are already in. " + "Is GMock thread safety enabled?"); + p = p->next_; + } p->next_ = this; next_ = ptr; } @@ -123,7 +128,12 @@ class linked_ptr_internal { if (next_ == this) return true; linked_ptr_internal const* p = next_; - while (p->next_ != this) p = p->next_; + while (p->next_ != this) { + assert(p->next_ != next_ && + "Trying to depart() a linked ring we are not in. " + "Is GMock thread safety enabled?"); + p = p->next_; + } p->next_ = next_; return false; } -- cgit v1.2.1