[PR #13778] Fix a crash on exit with the command palette open #29774

Closed
opened 2026-01-31 09:36:48 +00:00 by claunia · 0 comments
Owner

Original Pull Request: https://github.com/microsoft/terminal/pull/13778

State: closed
Merged: Yes


Fixes MSFT:38775539
Might also fix MSFT:38614563

Looking at this code should be pretty clear what's going on. On exit, the XAML root is already nulled out. But here, we're just yolo'ing and assuming it exists (why wouldn't it). So yea. This is like weirdly high percent of crashes internally, but that's not from real users. Real users, I suspect hit this as like .3% of our crashes. Not zero, but low.

  • tested manually

May also be related to...

  • MSFT:40602905
  • MSFT:40602904
  • MSFT:40412800
  • MSFT:35213459 <---has links
**Original Pull Request:** https://github.com/microsoft/terminal/pull/13778 **State:** closed **Merged:** Yes --- Fixes MSFT:38775539 Might also fix MSFT:38614563 Looking at this code should be pretty clear what's going on. On exit, the XAML root is already nulled out. But here, we're just yolo'ing and assuming it exists (why wouldn't it). So yea. This is like weirdly high percent of crashes internally, but that's not from real users. Real users, I suspect hit this as like .3% of our crashes. Not zero, but _low_. * [x] tested manually <hr> May also be related to... * MSFT:40602905 * MSFT:40602904 * MSFT:40412800 * MSFT:35213459 <---has links
claunia added the pull-request label 2026-01-31 09:36:48 +00:00
Sign in to join this conversation.
No Label pull-request
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#29774