[PATCH] D57924: [safestack] Move sysconf(_SC_PAGESIZE) back where it was before r351506

Vitaly Buka via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 7 14:11:50 PST 2019


vitalybuka added a comment.

In D57924#1389664 <https://reviews.llvm.org/D57924#1389664>, @vlad.tsyrklevich wrote:

> Alternatively, we could have a static buffer we use as a 'temporary' safe stack while initialization runs. I'm not really set on that idea, but it seems cleaner and like it may be more future proof in case another syscall we use (like getrlimit() right now) is intercepted. What do you think?


Hm, but buffer needs to be large, we don't know RLIMIT_STACK


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D57924/new/

https://reviews.llvm.org/D57924





More information about the llvm-commits mailing list