[compiler-rt] [scudo] Update secondary cache time-based release logic (PR #107507)
via llvm-commits
llvm-commits at lists.llvm.org
Mon Sep 16 14:50:33 PDT 2024
================
@@ -321,9 +322,10 @@ class MapAllocatorCache {
}
CachedBlock PrevEntry = Quarantine[QuarantinePos];
Quarantine[QuarantinePos] = Entry;
- if (OldestTime == 0)
- OldestTime = Entry.Time;
Entry = PrevEntry;
+ // Update the entry time to reflect the time that the
+ // quarantined memory is placed in the Entries array
+ Entry.Time = Time;
----------------
ChiaHungDuan wrote:
I noticed that we can't just change the time. Suppose the entry has been released then now we just mark it as unreleased again. This will cause some issue in MTE.
https://github.com/llvm/llvm-project/pull/107507
More information about the llvm-commits
mailing list