[PATCH] D41664: Remove test which assumed array cookies can't be poisoned when using an operator new defined in a class

Kostya Serebryany via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 16 15:19:16 PST 2018


kcc added a comment.

> How about this:
>  A `-fsanitize-address-poison-all-array-new` or similar (it would be all *except* placement new... Haven't got a better name, though).
>  That way, a user would be able to poison more array-new operators than the current solution. But we wouldn't break any legal C++ code.

Yes, I think this has to be a separate compile-time flag, off by default, at least initially. 
`-fsanitize-address-poison-all-array-new` is indeed a non-perfect name. 
How about `-fsanitize-address-poison-class member-array-new-cookie` (or some such) to be more explicit?


Repository:
  rCRT Compiler Runtime

https://reviews.llvm.org/D41664





More information about the llvm-commits mailing list