Skip to content

Fix a race condition around Open/CloseClipboard#19297

Merged
lhecker merged 1 commit intomainfrom
dev/lhecker/clipboard-crash
Aug 29, 2025
Merged

Fix a race condition around Open/CloseClipboard#19297
lhecker merged 1 commit intomainfrom
dev/lhecker/clipboard-crash

Conversation

@lhecker
Copy link
Member

@lhecker lhecker commented Aug 29, 2025

tl;dr: Open/CloseClipboard are surprisingly not thread-safe.

Validation Steps Performed

  • Copy a large amount of text (>1MB)
  • Run edit.exe
  • Press and hold Ctrl+Shift+V
  • Doesn't crash ✅

@lhecker lhecker added Issue-Bug It either shouldn't be doing this or needs an investigation. Product-Terminal The new Windows Terminal. Priority-2 A description (P2) labels Aug 29, 2025
@github-project-automation github-project-automation bot moved this to To Cherry Pick in 1.23 Servicing Pipeline Aug 29, 2025
@github-project-automation github-project-automation bot moved this to To Cherry Pick in 1.24 Servicing Pipeline Aug 29, 2025
@lhecker lhecker merged commit 5899343 into main Aug 29, 2025
19 checks passed
@lhecker lhecker deleted the dev/lhecker/clipboard-crash branch August 29, 2025 18:20
@DHowett DHowett moved this from To Cherry Pick to To Consider in 1.23 Servicing Pipeline Sep 5, 2025
@DHowett DHowett moved this from To Cherry Pick to Cherry Picked in 1.24 Servicing Pipeline Sep 5, 2025
DHowett pushed a commit that referenced this pull request Sep 5, 2025
tl;dr: Open/CloseClipboard are surprisingly not thread-safe.

## Validation Steps Performed
* Copy a large amount of text (>1MB)
* Run `edit.exe`
* Press and hold Ctrl+Shift+V
* Doesn't crash ✅

(cherry picked from commit 5899343)
Service-Card-Id: PVTI_lADOAF3p4s4BBcTlzgeLIjw
Service-Version: 1.24
@DHowett DHowett moved this from Cherry Picked to Shipped in 1.24 Servicing Pipeline Feb 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Issue-Bug It either shouldn't be doing this or needs an investigation. Priority-2 A description (P2) Product-Terminal The new Windows Terminal.

Projects

Status: To Consider

Development

Successfully merging this pull request may close these issues.

2 participants