[PATCH] Poison all slabs on BumpPtrAllocator::Reset

Pete Cooper via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 11 13:32:55 PST 2016


Committed as r257388 after rerunning ‘make check-all’ with this applied and asan enabled.

Thanks,
Pete

> On Jan 8, 2016, at 5:15 PM, Pete Cooper via llvm-commits <llvm-commits at lists.llvm.org> wrote:
> 
> Thanks for the CC Alexey.
> 
> LGTM.  I don’t know how I missed this case.  I’ll apply it now and commit if it passes.  Thanks b17 c0de.
> 
> Pete
>> On Jan 8, 2016, at 5:09 PM, Alexey Samsonov <vonosmas at gmail.com <mailto:vonosmas at gmail.com>> wrote:
>> 
>> +Pete, Justin who landed the original poisoning for BumpPtrAllocator.
>> 
>> On Fri, Jan 8, 2016 at 3:44 PM, David Blaikie <dblaikie at gmail.com <mailto:dblaikie at gmail.com>> wrote:
>> bump/pass
>> 
>> Alexey - is this the right asanification here?
>> 
>> On Fri, Dec 18, 2015 at 1:07 AM, b17 c0de via llvm-commits <llvm-commits at lists.llvm.org <mailto:llvm-commits at lists.llvm.org>> wrote:
>> BumpPtrAllocator::Reset should also poison the first slab which doesn't get deallocated. Please apply.
>> 
>> _______________________________________________
>> llvm-commits mailing list
>> llvm-commits at lists.llvm.org <mailto:llvm-commits at lists.llvm.org>
>> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits <http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits>
>> 
>> 
>> 
>> 
>> 
>> -- 
>> Alexey Samsonov
>> vonosmas at gmail.com <mailto:vonosmas at gmail.com>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160111/c516c5e9/attachment.html>


More information about the llvm-commits mailing list