Skip to content

Conversation

@pvollan
Copy link
Contributor

@pvollan pvollan commented Jan 21, 2025

@pvollan pvollan requested a review from brentfulgham as a code owner January 21, 2025 01:48
@pvollan pvollan self-assigned this Jan 21, 2025
@pvollan pvollan added the WebKit Process Model Bugs related to WebKit's multi-process architecture label Jan 21, 2025
@webkit-early-warning-system
Copy link
Collaborator

Starting EWS tests for c54c875. Live statuses available at the PR page, #39306

@webkit-ews-buildbot webkit-ews-buildbot added the merging-blocked Applied to prevent a change from being merged label Jan 21, 2025
@geoffreygaren
Copy link
Contributor

Nice!

@pvollan pvollan added unsafe-merge-queue Applied to send a pull request to merge-queue, but skip building and testing and removed merging-blocked Applied to prevent a change from being merged labels Feb 1, 2025
https://bugs.webkit.org/show_bug.cgi?id=286279
rdar://143139266

Reviewed by Sihui Liu.

We currently block all messages to the service, but we should also block the Mach lookup.

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

Canonical link: https://commits.webkit.org/289675@main
@webkit-commit-queue webkit-commit-queue force-pushed the eng/Block-Mach-lookup-in-the-WebContent-sandbox branch from c54c875 to aeb86cf Compare February 1, 2025 20:01
@webkit-commit-queue
Copy link
Collaborator

Committed 289675@main (aeb86cf): https://commits.webkit.org/289675@main

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

@webkit-commit-queue webkit-commit-queue merged commit aeb86cf into WebKit:main Feb 1, 2025
@webkit-commit-queue webkit-commit-queue removed the unsafe-merge-queue Applied to send a pull request to merge-queue, but skip building and testing label Feb 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

WebKit Process Model Bugs related to WebKit's multi-process architecture

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants