[compiler-rt] [scudo] do not store size inside ring buffer (PR #74541)
Christopher Ferris via llvm-commits
llvm-commits at lists.llvm.org
Fri Dec 8 17:16:35 PST 2023
================
@@ -1408,17 +1409,19 @@ class Allocator {
size_t &NextErrorReport,
uintptr_t FaultAddr,
const StackDepot *Depot,
- const char *RingBufferPtr) {
+ const char *RingBufferPtr,
+ size_t RingBufferSize) {
auto *RingBuffer =
reinterpret_cast<const AllocationRingBuffer *>(RingBufferPtr);
- if (!RingBuffer || RingBuffer->Size == 0)
+ size_t RingBufferElements = ringBufferElementsFromBytes(RingBufferSize);
+ if (!RingBuffer || RingBufferElements == 0)
----------------
cferris1000 wrote:
Sounds reasonable to me.
https://github.com/llvm/llvm-project/pull/74541
More information about the llvm-commits
mailing list