This is an archive of the discontinued LLVM Phabricator instance.

ASan+operator new[]: Fix operator new[] cookie poisoning
ClosedPublic

Authored by filcab on Dec 15 2017, 9:59 AM.

Details

Summary

The C++ Itanium ABI says:
No cookie is required if the new operator being used is ::operator new[](size_t, void*).

We should only avoid poisoning the cookie if we're calling this
operator, not others. This is dealt with before the call to
InitializeArrayCookie.

Diff Detail

Repository
rL LLVM

Event Timeline

filcab created this revision.Dec 15 2017, 9:59 AM
rjmccall accepted this revision.Dec 15 2017, 6:51 PM

LGTM.

This revision is now accepted and ready to land.Dec 15 2017, 6:51 PM
This revision was automatically updated to reflect the committed changes.