Skip to content

Remove access to purplebuddy.sentinel from web process's sandbox#26423

Merged
webkit-commit-queue merged 1 commit into
WebKit:mainfrom
szewai:eng/Remove-access-to-purplebuddy-sentinel-from-web-processs-sandbox
Mar 26, 2024
Merged

Remove access to purplebuddy.sentinel from web process's sandbox#26423
webkit-commit-queue merged 1 commit into
WebKit:mainfrom
szewai:eng/Remove-access-to-purplebuddy-sentinel-from-web-processs-sandbox

Conversation

@szewai
Copy link
Copy Markdown
Contributor

@szewai szewai commented Mar 25, 2024

@szewai szewai self-assigned this Mar 25, 2024
@szewai szewai added the New Bugs Unclassified bugs are placed in this component until the correct component can be determined. label Mar 25, 2024
@webkit-early-warning-system
Copy link
Copy Markdown
Collaborator

webkit-early-warning-system commented Mar 25, 2024

@szewai szewai marked this pull request as ready for review March 25, 2024 19:11
Copy link
Copy Markdown
Contributor

@pvollan pvollan left a comment

Choose a reason for hiding this comment

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

Nice! R=me.

@szewai szewai force-pushed the eng/Remove-access-to-purplebuddy-sentinel-from-web-processs-sandbox branch from e7eb58e to 3f1e925 Compare March 26, 2024 04:33
@szewai szewai added the merge-queue Applied to send a pull request to merge-queue label Mar 26, 2024
https://bugs.webkit.org/show_bug.cgi?id=271659
rdar://125361229

Reviewed by Per Arne Vollan.

Web process does not need access to the purplebuddy semaphore.

* Source/WebKit/Resources/SandboxProfiles/ios/com.apple.WebKit.WebContent.sb.in:

Canonical link: https://commits.webkit.org/276677@main
@webkit-commit-queue webkit-commit-queue force-pushed the eng/Remove-access-to-purplebuddy-sentinel-from-web-processs-sandbox branch from 3f1e925 to d03ff2b Compare March 26, 2024 07:19
@webkit-commit-queue
Copy link
Copy Markdown
Collaborator

Committed 276677@main (d03ff2b): https://commits.webkit.org/276677@main

Reviewed commits have been landed. Closing PR #26423 and removing active labels.

@webkit-commit-queue webkit-commit-queue merged commit d03ff2b into WebKit:main Mar 26, 2024
@webkit-commit-queue webkit-commit-queue removed the merge-queue Applied to send a pull request to merge-queue label Mar 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

New Bugs Unclassified bugs are placed in this component until the correct component can be determined.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants