[PATCH] D21902: [compiler-rt] Fix TLS resource leaking in unittest

Etienne Bergeron via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 30 12:37:05 PDT 2016


etienneb updated this revision to Diff 62401.
etienneb added a comment.

nits


http://reviews.llvm.org/D21902

Files:
  lib/sanitizer_common/tests/sanitizer_posix_test.cc

Index: lib/sanitizer_common/tests/sanitizer_posix_test.cc
===================================================================
--- lib/sanitizer_common/tests/sanitizer_posix_test.cc
+++ lib/sanitizer_common/tests/sanitizer_posix_test.cc
@@ -56,6 +56,7 @@
   EXPECT_TRUE(destructor_executed);
   SpawnThread(GetPthreadDestructorIterations() + 1);
   EXPECT_FALSE(destructor_executed);
+  ASSERT_EQ(0, pthread_key_delete(key));
 }
 
 TEST(SanitizerCommon, IsAccessibleMemoryRange) {


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D21902.62401.patch
Type: text/x-patch
Size: 480 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160630/6075bded/attachment.bin>


More information about the llvm-commits mailing list