[PATCH] D72364: [scudo][standalone] Modify malloc_{enable,disable} wrt fork

Wed Jan 8 11:18:06 PST 2020

cryptoad added a comment.

In D72364#1810436 <https://reviews.llvm.org/D72364#1810436>, @eugenis wrote:

> Could we do this exact same thing in Android? I find this behavior (auto-enabling the allocator after fork) unexpected, and it makes the interface even more confusing then it is now.

We discussed it with cferris@ and the two behaviors that were deemed acceptable were:

- enable in child post fork
- block fork

A test is being added with https://android-review.googlesource.com/c/platform/system/memory/libmemunreachable/+/1202475.

I went with enable in child post fork because it was straightforward but I can go the other way as well.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D72364