[PATCH] D15008: [sanitizer] Fix a crash in SizeClassAllocator32 with an out-of-range pointer

Kostya Serebryany via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 8 03:11:30 PST 2015


kcc added a comment.

In http://reviews.llvm.org/D15008#304630, @dvyukov wrote:

> Kostya, I think we should remove kSpaceBeg and replace it with zero. What do you think?


You mean, only SizeClassAllocator32::kSpaceBeg, right? 
Why?

Theoretically, we may gain something by making kSpaceBeg non-zero.
Before we actually do it, SizeClassAllocator32::kSpaceBeg is kind of useless, but it also doesn't harm much.


http://reviews.llvm.org/D15008





More information about the llvm-commits mailing list