Page MenuHomePhabricator

[docs][scudo] Update Scudo documentation
ClosedPublic

Authored by cryptoad on Apr 9 2021, 2:15 PM.

Details

Summary

Update the Scudo document to align with the standalone version.
Add some more verbiage about the various component of the
allocator, rework a bit everything.
The build instructions have been updated.
The options and their default values have been updated, and
the mallopt ones have been added.

Diff Detail

Event Timeline

cryptoad created this revision.Apr 9 2021, 2:15 PM
cryptoad requested review of this revision.Apr 9 2021, 2:15 PM
Herald added a project: Restricted Project. · View Herald TranscriptApr 9 2021, 2:15 PM
hctim accepted this revision.Apr 9 2021, 3:26 PM

It would be nice to include some details about MTE support (like the header size is 16 bytes rather than 8 bytes, etc). But we can make those changes afterwards.

llvm/docs/ScudoHardenedAllocator.rst
13

nit, push url onto newline to help with line length?

16

Android 11

20–22

nit - start on newline?

69

if

This revision is now accepted and ready to land.Apr 9 2021, 3:26 PM
cryptoad updated this revision to Diff 336581.Apr 9 2021, 4:45 PM
cryptoad marked 4 inline comments as done.

Addressing review comments.

This revision was automatically updated to reflect the committed changes.