Skip to content

Fix crash in start_display_link#50875

Merged
reflectronic merged 1 commit intomainfrom
fix-displaylink-crash
Mar 5, 2026
Merged

Fix crash in start_display_link#50875
reflectronic merged 1 commit intomainfrom
fix-displaylink-crash

Conversation

@reflectronic
Copy link
Copy Markdown
Member

@reflectronic reflectronic commented Mar 5, 2026

Fixes ZED-5G8

If DisplayLink::new fails, frame_requests is dropped . It is not valid to destroy a DispatchSource that is not resume()d. So, ensure we call resume() before there's a chance for anything to fail.

Release Notes:

  • (preview only) Fixed a crash that could occur on macOS when changing monitor configurations

@cla-bot cla-bot bot added the cla-signed The user has signed the Contributor License Agreement label Mar 5, 2026
@zed-community-bot zed-community-bot bot added the staff Pull requests authored by a current member of Zed staff label Mar 5, 2026
@reflectronic reflectronic enabled auto-merge (squash) March 5, 2026 21:52
@reflectronic
Copy link
Copy Markdown
Member Author

/cherry-pick preview

@reflectronic reflectronic merged commit 41cfb82 into main Mar 5, 2026
40 checks passed
@reflectronic reflectronic deleted the fix-displaylink-crash branch March 5, 2026 21:54
github-actions bot pushed a commit that referenced this pull request Mar 5, 2026
Fixes ZED-5G8

If `DisplayLink::new` fails, `frame_requests` is dropped . It is not
valid to destroy a DispatchSource that is not `resume()`d. So, ensure we
call `resume()` before there's a chance for anything to fail.

Release Notes:

- Fixed a crash that could occur on macOS when changing monitor
configurations
zed-zippy bot added a commit that referenced this pull request Mar 5, 2026
…50878)

Cherry-pick of #50875 to preview

----
Fixes ZED-5G8

If `DisplayLink::new` fails, `frame_requests` is dropped . It is not
valid to destroy a DispatchSource that is not `resume()`d. So, ensure we
call `resume()` before there's a chance for anything to fail.

Release Notes:

- Fixed a crash that could occur on macOS when changing monitor
configurations

Co-authored-by: John Tur <john-tur@outlook.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla-signed The user has signed the Contributor License Agreement staff Pull requests authored by a current member of Zed staff

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant