Skip to content

Add support for the Scudo sanitizer. #2438

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Apr 29, 2020
Merged

Conversation

Lukasa
Copy link
Contributor

@Lukasa Lukasa commented Dec 4, 2019

The Swift PR swiftlang/swift#28538 adds support for the Scudo hardened
allocator to swiftc. This patch adds support for it to Swift Package
Manager, allowing the Package Manager to correctly plumb Scudo through
the entire build pipeline.

See the aforementioned PR for more discussion on the value of supporting
Scudo.

Note that the tests will fail until swiftlang/swift#28538 lands.

@Lukasa Lukasa requested a review from aciidgh as a code owner December 4, 2019 17:21
@Lukasa
Copy link
Contributor Author

Lukasa commented Apr 27, 2020

@swift-ci please test

@Lukasa
Copy link
Contributor Author

Lukasa commented Apr 27, 2020

I've brought this patch back up-to-date with master, as the scudo support did in fact land. It'd be nice if we could back port this to 5.3, if it's not too late to do so.

The Swift PR swiftlang/swift#28538 adds support for the Scudo hardened
allocator to swiftc. This patch adds support for it to Swift Package
Manager, allowing the Package Manager to correctly plumb Scudo through
the entire build pipeline.

See the aforementioned PR for more discussion on the value of supporting
Scudo.
@Lukasa
Copy link
Contributor Author

Lukasa commented Apr 27, 2020

@swift-ci please smoke test

@Lukasa
Copy link
Contributor Author

Lukasa commented Apr 27, 2020

@swift-ci please test

1 similar comment
@Lukasa
Copy link
Contributor Author

Lukasa commented Apr 28, 2020

@swift-ci please test

Copy link
Contributor

@abertelrud abertelrud left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Makes sense to me. The flag was merged into this compiler a long time ago. There is a FIXME in the code about detecting mutually incompatible sanitizers, but since that's not yet implemented, there is nothing this PR can/should do specially there yet.

@abertelrud abertelrud merged commit 8064187 into swiftlang:master Apr 29, 2020
@abertelrud
Copy link
Contributor

Merged. Please open a separate PR for the 5.3, which Rick (release manager for SwiftPM in 5.3) will need to approve.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants