[PATCH] D70908: [scudo][standalone] Add chunk ownership function

Mitch Phillips via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 2 13:30:30 PST 2019


hctim accepted this revision.
hctim added a comment.
This revision is now accepted and ready to land.

In D70908#1765917 <https://reviews.llvm.org/D70908#1765917>, @cryptoad wrote:

> I think that not claiming ownership of a corrupted chunk is something we can live with.
>  The ownership concept for tcmalloc has strong requirements, which I think are fulfilled by this implementation.


Ack - would you mind mentioning this decision in the function description comment?

Otherwise LGTM.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D70908/new/

https://reviews.llvm.org/D70908





More information about the llvm-commits mailing list