[compiler-rt] r298195 - Change stack-use-after-return.cc to respect PTHREAD_STACK_MIN before calling pthread_attr_setstacksize. To investigate <https://reviews.llvm.org/D30267>. NFC.

Kuba Mracek via llvm-commits llvm-commits at lists.llvm.org
Sat Mar 18 14:35:12 PDT 2017


Author: kuba.brecka
Date: Sat Mar 18 16:35:12 2017
New Revision: 298195

URL: http://llvm.org/viewvc/llvm-project?rev=298195&view=rev
Log:
Change stack-use-after-return.cc to respect PTHREAD_STACK_MIN before calling pthread_attr_setstacksize. To investigate <https://reviews.llvm.org/D30267>. NFC.


Modified:
    compiler-rt/trunk/test/asan/TestCases/Posix/stack-use-after-return.cc

Modified: compiler-rt/trunk/test/asan/TestCases/Posix/stack-use-after-return.cc
URL: http://llvm.org/viewvc/llvm-project/compiler-rt/trunk/test/asan/TestCases/Posix/stack-use-after-return.cc?rev=298195&r1=298194&r2=298195&view=diff
==============================================================================
--- compiler-rt/trunk/test/asan/TestCases/Posix/stack-use-after-return.cc (original)
+++ compiler-rt/trunk/test/asan/TestCases/Posix/stack-use-after-return.cc Sat Mar 18 16:35:12 2017
@@ -68,7 +68,12 @@ int main(int argc, char **argv) {
   pthread_attr_t attr;
   pthread_attr_init(&attr);
   if (kStackSize > 0) {
-    int ret = pthread_attr_setstacksize(&attr, kStackSize);
+    size_t desired_stack_size = kStackSize;
+    if (desired_stack_size < PTHREAD_STACK_MIN) {
+      desired_stack_size = PTHREAD_STACK_MIN;
+    }
+
+    int ret = pthread_attr_setstacksize(&attr, desired_stack_size);
     if (ret != 0) {
       fprintf(stderr, "pthread_attr_setstacksize returned %d\n", ret);
       abort();
@@ -81,9 +86,9 @@ int main(int argc, char **argv) {
       abort();
     }
 
-    if (stacksize_check != kStackSize) {
+    if (stacksize_check != desired_stack_size) {
       fprintf(stderr, "Unable to set stack size to %d, the stack size is %d.\n",
-              kStackSize, stacksize_check);
+              desired_stack_size, stacksize_check);
       abort(); 
     }
   }




More information about the llvm-commits mailing list