Skip to content

[Backtracing] Security improvements. #65067

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 3 commits into from
Apr 14, 2023

Conversation

al45tair
Copy link
Contributor

Use task_read_for_pid() rather than having the crashing program pass its own task port through. This opts us in to additional OS security measures surrounding the use of this call.

rdar://107362003

@al45tair al45tair requested a review from a team as a code owner April 11, 2023 16:30
@al45tair
Copy link
Contributor Author

This is a cherry pick of #65066.

@al45tair al45tair requested a review from mikeash April 11, 2023 16:31
@al45tair
Copy link
Contributor Author

@swift-ci Please test

Use `task_read_for_pid()` rather than having the crashing program pass its
own task port through.  This opts us in to additional OS security measures
surrounding the use of this call.

rdar://107362003
@al45tair al45tair force-pushed the eng/PR-107362003-5.9 branch from d8fe096 to 981eb75 Compare April 12, 2023 11:50
@al45tair
Copy link
Contributor Author

@swift-ci Please test

Using the hardened runtime for tests doesn't work.

rdar://107362003
@al45tair
Copy link
Contributor Author

@swift-ci Please test

1 similar comment
@al45tair
Copy link
Contributor Author

@swift-ci Please test

For some reason it wants two blank lines instead of one here.

rdar://107362003
@al45tair
Copy link
Contributor Author

@swift-ci Please test

@tbkka tbkka requested a review from tomerd April 14, 2023 17:04
@al45tair al45tair merged commit 9194aa4 into swiftlang:release/5.9 Apr 14, 2023
@AnthonyLatsis AnthonyLatsis added the 🍒 release cherry pick Flag: Release branch cherry picks label May 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🍒 release cherry pick Flag: Release branch cherry picks
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants