This is an archive of the discontinued LLVM Phabricator instance.

[scudo] Always exclude Tag 0
ClosedPublic

Authored by vitalybuka on May 25 2021, 6:41 PM.

Details

Summary

prepareTaggedChunk uses Tag 0 for header.

Android already PR_MTE_TAG_MASK to 0xfffe,
but with the patch we will not need to deppend
on the system configuration.

Diff Detail

Event Timeline

vitalybuka requested review of this revision.May 25 2021, 6:41 PM
vitalybuka created this revision.
Herald added a project: Restricted Project. · View Herald TranscriptMay 25 2021, 6:41 PM
Herald added a subscriber: Restricted Project. · View Herald Transcript
pcc added a comment.May 25 2021, 6:53 PM

There are probably going to be more callers of selectRandomTag and I don't think we want to have to deal with tag 0 in any of them. Could we just or 1 into the mask in selectRandomTag?

Could we just or 1 into the mask in selectRandomTag?

done

pcc accepted this revision.Jun 3 2021, 8:15 PM

LGTM

This revision is now accepted and ready to land.Jun 3 2021, 8:15 PM
This revision was automatically updated to reflect the committed changes.