commit b134ffa00ce2a4cbf7f48ac029e31c3e6e23ff6b
author Kostya Serebryany <kcc@google.com> Tue Jul 17 07:20:13 2012 +0000
committer Kostya Serebryany <kcc@google.com> Tue Jul 17 07:20:13 2012 +0000
tree 9c98975bf5c1e4d968f099357fb5c9c3560af035
parent bca62bd769f7d736c0e1953c5682d176cd1090a4

[asan] get rid of the last operator new call in asan rtl

git-svn-id: https://llvm.org/svn/llvm-project/compiler-rt/trunk@160347 91177308-0d34-0410-b5e6-96231b3b80d8

lib/asan/asan_rtl.cc

diff --git a/lib/asan/asan_rtl.cc b/lib/asan/asan_rtl.cc
index 6f71708..b3a7b16 100644
--- a/lib/asan/asan_rtl.cc
+++ b/lib/asan/asan_rtl.cc
@@ -48,7 +48,7 @@
 }
 
 void CheckFailed(const char *file, int line, const char *cond, u64 v1, u64 v2) {
-  AsanReport("AddressSanitizer CHECK failed: %s:%d \"%s\" (%zx, %zx)\n",
+  AsanReport("AddressSanitizer CHECK failed: %s:%d \"%s\" (0x%zx, 0x%zx)\n",
              file, line, cond, (uptr)v1, (uptr)v2);
   PRINT_CURRENT_STACK();
   ShowStatsAndAbort();

lib/asan/asan_thread.cc

diff --git a/lib/asan/asan_thread.cc b/lib/asan/asan_thread.cc
index 2f27732..a46ae28 100644
--- a/lib/asan/asan_thread.cc
+++ b/lib/asan/asan_thread.cc
@@ -26,6 +26,9 @@
       malloc_storage_(x),
       stats_(x) { }
 
+static AsanLock mu_for_thread_summary(LINKER_INITIALIZED);
+static LowLevelAllocator allocator_for_thread_summary(LINKER_INITIALIZED);
+
 AsanThread *AsanThread::Create(u32 parent_tid, thread_callback_t start_routine,
                                void *arg, AsanStackTrace *stack) {
   uptr size = RoundUpTo(sizeof(AsanThread), kPageSize);
@@ -33,7 +36,15 @@
   thread->start_routine_ = start_routine;
   thread->arg_ = arg;
 
-  AsanThreadSummary *summary = new AsanThreadSummary(parent_tid, stack);
+  const uptr kSummaryAllocSize = 1024;
+  CHECK_LE(sizeof(AsanThreadSummary), kSummaryAllocSize);
+  AsanThreadSummary *summary;
+  {
+    ScopedLock lock(&mu_for_thread_summary);
+    summary = (AsanThreadSummary*)
+        allocator_for_thread_summary.Allocate(kSummaryAllocSize);
+  }
+  summary->Init(parent_tid, stack);
   summary->set_thread(thread);
   thread->set_summary(summary);
 

lib/asan/asan_thread.h

diff --git a/lib/asan/asan_thread.h b/lib/asan/asan_thread.h
index 4d356cc..9a032fe 100644
--- a/lib/asan/asan_thread.h
+++ b/lib/asan/asan_thread.h
@@ -18,6 +18,7 @@
 #include "asan_internal.h"
 #include "asan_stack.h"
 #include "asan_stats.h"
+#include "sanitizer_common/sanitizer_libc.h"
 
 namespace __asan {
 
@@ -30,12 +31,12 @@
 class AsanThreadSummary {
  public:
   explicit AsanThreadSummary(LinkerInitialized) { }  // for T0.
-  AsanThreadSummary(u32 parent_tid, AsanStackTrace *stack)
-      : parent_tid_(parent_tid),
-        announced_(false) {
+  void Init(u32 parent_tid, AsanStackTrace *stack) {
+    parent_tid_ = parent_tid;
+    announced_ = false;
     tid_ = kInvalidTid;
     if (stack) {
-      stack_ = *stack;
+      internal_memcpy(&stack_, stack, sizeof(*stack));
     }
     thread_ = 0;
   }