Please use GitHub pull requests for new patches. Avoid migrating existing patches. Phabricator shutdown timeline

Differential D145407

[scudo] Shuffle the regions
ClosedPublic
Actions

Authored by Chia-hungDuan on Mar 6 2023, 11:13 AM.

Tags

Subscribers

Restricted Project

Details

Reviewers

cferris
cryptoad

Commits

rG849da0320253: [scudo] Shuffle the regions

Summary

Shuffle the regions' base address so that the layout of all regions is
less predictable.

Diff Detail

Repository: rG LLVM Github Monorepo

Event Timeline

Chia-hungDuan created this revision.Mar 6 2023, 11:13 AM

Herald added a project: Restricted Project. · View Herald TranscriptMar 6 2023, 11:13 AM

Herald added subscribers: yaneury, supersymetrie, Enna1. · View Herald Transcript

Chia-hungDuan requested review of this revision.Mar 6 2023, 11:13 AM

Herald added a project: Restricted Project. · View Herald TranscriptMar 6 2023, 11:13 AM

Herald added a subscriber: Restricted Project. · View Herald Transcript

Chia-hungDuan added reviewers: cferris, cryptoad.Mar 6 2023, 11:14 AM

I like the idea of randomizing the regions' base!

This revision is now accepted and ready to land.Mar 6 2023, 11:26 AM

LGTM.

Harbormaster completed remote builds in B217655: Diff 502738.Mar 6 2023, 3:21 PM

Closed by commit rG849da0320253: [scudo] Shuffle the regions (authored by Chia-hungDuan). · Explain WhyMar 8 2023, 2:59 PM

This revision was automatically updated to reflect the committed changes.

Chia-hungDuan added a commit: rG849da0320253: [scudo] Shuffle the regions.

Revision Contents
Changeset List

Path

Size

compiler-rt/

lib/

scudo/

standalone/

11 lines

Diff 503533

compiler-rt/lib/scudo/standalone/primary64.h

Loading...