From: Dmitry Vyukov <dvyukov@google.com>
Date: Fri, 6 Jun 2014 15:56:08 +0000 (+0000)
Subject: tsan: fix out-of-bounds access in Go runtime
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=75f5cf657eb8036bcd2d49df9933568e6fa3df43;p=platform%2Fupstream%2Fllvm.git

tsan: fix out-of-bounds access in Go runtime
FuncEntry can resize the shadow stack, while "thr->shadow_stack_pos[0] = pc" writes out-of-bounds.

llvm-svn: 210349
---

diff --git a/compiler-rt/lib/tsan/rtl/tsan_rtl.cc b/compiler-rt/lib/tsan/rtl/tsan_rtl.cc
index fe95971..add6bd7 100644
--- a/compiler-rt/lib/tsan/rtl/tsan_rtl.cc
+++ b/compiler-rt/lib/tsan/rtl/tsan_rtl.cc
@@ -423,13 +423,11 @@ void ForkChildAfter(ThreadState *thr, uptr pc) {
 u32 CurrentStackId(ThreadState *thr, uptr pc) {
   if (thr->shadow_stack_pos == 0)  // May happen during bootstrap.
     return 0;
-  if (pc) {
-    thr->shadow_stack_pos[0] = pc;
-    thr->shadow_stack_pos++;
-  }
+  if (pc != 0)
+    FuncEntry(thr, pc);  // can resize the shadow stack
   u32 id = StackDepotPut(thr->shadow_stack,
                          thr->shadow_stack_pos - thr->shadow_stack);
-  if (pc)
+  if (pc != 0)
     thr->shadow_stack_pos--;
   return id;
 }