8000 mac: release the IPC server receive right by SomeoneToIgnore · Pull Request #249 · rust-minidump/minidump-writer · GitHub
[go: up one dir, main page]

Skip to content

mac: release the IPC server receive right - #249

Merged
schopin-mozilla merged 1 commit into
rust-minidump:mainfrom
SomeoneToIgnore:fix-quit-panic
Oct 26, 2026
Merged

schopin-mozilla merged 1 commit into
rust-minidump:mainfrom
SomeoneToIgnore:fix-quit-panic

Conversation

@SomeoneToIgnore
Copy link
Copy Markdown
Contributor

bootstrap_check_in returns a receive right, but the IPC server drops it with mach_port_deallocate, which only releases send, send-once, or dead-name references.
This was found while investigating Zed’s EXC_GUARD / INVALID_RIGHT crash on quit.

bootstrap_check_in returns a receive right, but mach_port_deallocate only releases send, send-once, or dead-name references.
Release the receive right with mach_port_mod_refs instead.

Add a regression test that checks the port name is removed after dropping a server with no send references.

Refs: zed-industries/zed#57664
@schopin-mozilla schopin-mozilla left a comment
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

Comment thread crates/crash-context/src/mac/ipc.rs Resolved
@schopin-mozilla
schopin-mozilla merged commit 911b04e into rust-minidump:main Oct 26, 2026
20 checks passed
@SomeoneToIgnore
SomeoneToIgnore deleted the fix-quit-panic branch October 26, 2026 10:44
AurySepe pushed a commit to AurySepe/zed that referenced this pull request Oct 26, 2026
Closes zed-industries#57664

Updates `crash-context` transitive dependency (ergo the `patch` section)
to include rust-minidump/minidump-writer#249
fix: given the latest release was a few months ago, decided to use the
git SHA pointer.

Release Notes:

- Fixed "Zed quit unexpectedly" when quitting via ⌘ + Q
slgobinath pushed a commit to slgobinath/bench that referenced this pull request Nov 6, 2026
Closes zed-industries#57664

Updates `crash-context` transitive dependency (ergo the `patch` section)
to include rust-minidump/minidump-writer#249
fix: given the latest release was a few months ago, decided to use the
git SHA pointer.

Release Notes:

- Fixed "Zed quit unexpectedly" when quitting via ⌘ + Q

(cherry picked from commit 087340e)
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.

2 participants

0