[PATCH] D23589: Fix hang caused by memory corruption caused by insufficient alternate stack space for signal handlers.

bryant via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 22 16:26:23 PDT 2016


bryant updated this revision to Diff 68929.
bryant added a comment.

"'64K ought to be enough for anyone' -- bill gates " -- bryant.


Repository:
  rL LLVM

https://reviews.llvm.org/D23589

Files:
  lib/Support/Unix/Signals.inc

Index: lib/Support/Unix/Signals.inc
===================================================================
--- lib/Support/Unix/Signals.inc
+++ lib/Support/Unix/Signals.inc
@@ -127,7 +127,7 @@
 static stack_t OldAltStack;
 
 static void CreateSigAltStack() {
-  const size_t AltStackSize = MINSIGSTKSZ + 8192;
+  const size_t AltStackSize = MINSIGSTKSZ + 64 * 1024;
 
   // If we're executing on the alternate stack, or we already have an alternate
   // signal stack that we're happy with, there's nothing for us to do. Don't


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D23589.68929.patch
Type: text/x-patch
Size: 524 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160822/34960828/attachment.bin>


More information about the llvm-commits mailing list