[compiler-rt] [scudo] Handle failed mmap for allocation ring buffer (PR #71817)

Florian Mayer via llvm-commits llvm-commits at lists.llvm.org
Fri Nov 10 02:20:25 PST 2023


================
@@ -1281,7 +1281,8 @@ class Allocator {
 
   void storeSecondaryAllocationStackMaybe(const Options &Options, void *Ptr,
                                           uptr Size) {
-    if (!UNLIKELY(Options.get(OptionBit::TrackAllocationStacks)))
+    if (!UNLIKELY(Options.get(OptionBit::TrackAllocationStacks)) ||
+        RawRingBuffer == nullptr)
----------------
fmayer wrote:

I would rather not change the external behavior without good reason. It is currently certainly possible to turn it of and back on.

https://github.com/llvm/llvm-project/pull/71817


More information about the llvm-commits mailing list