Do not call disable / enable on null depot

[fmayer]

depot can be null if allocation_ring_buffer_size=0

[llvmbot]

@llvm/pr-subscribers-compiler-rt-sanitizer

Author: Florian Mayer (fmayer)

Changes

depot can be null if allocation_ring_buffer_size=0

---

Full diff: https://github.com/llvm/llvm-project/pull/82542.diff

1 Files Affected:

• (modified) compiler-rt/lib/scudo/standalone/combined.h (+4-2)

diff --git a/compiler-rt/lib/scudo/standalone/combined.h b/compiler-rt/lib/scudo/standalone/combined.h
index 080ba42ad44497..f3c3d757c9f128 100644
--- a/compiler-rt/lib/scudo/standalone/combined.h
+++ b/compiler-rt/lib/scudo/standalone/combined.h
@@ -688,12 +688,14 @@ class Allocator {
     Quarantine.disable();
     Primary.disable();
     Secondary.disable();
-    Depot->disable();
+    if (Depot)
+      Depot->disable();
   }
 
   void enable() NO_THREAD_SAFETY_ANALYSIS {
     initThreadMaybe();
-    Depot->enable();
+    if (Depot)
+      Depot->enable();
     Secondary.enable();
     Primary.enable();
     Quarantine.enable();

[dwblaikie]

reposting here rather than on the commit itself - should this change have test covergae? (or was it fixing a pre-existing/failing test?)