[PR #14653] Check for null lParam in WM_SETTINGSCHANGE #30192

Open
opened 2026-01-31 09:39:15 +00:00 by claunia · 0 comments
Owner

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

State: closed
Merged: Yes


Fix access violation on lParam when constructing wstring.

Per the WM_SETTINGSCHANGE docs, NULL is a valid value for lParam. A
null lParam was observed when using the 'Switch User' feature of Windows
while developing the Terminal app.

Validation Steps Performed

Reproduced the scenario w/ the check in place and verified no crash.

Closes #14652

**Original Pull Request:** https://github.com/microsoft/terminal/pull/14653 **State:** closed **Merged:** Yes --- Fix access violation on lParam when constructing wstring. Per the [WM_SETTINGSCHANGE docs], NULL is a valid value for lParam. A null lParam was observed when using the 'Switch User' feature of Windows while developing the Terminal app. ## Validation Steps Performed Reproduced the scenario w/ the check in place and verified no crash. Closes #14652 [WM_SETTINGSCHANGE docs]: https://learn.microsoft.com/en-us/windows/win32/winmsg/wm-settingchange
claunia added the pull-request label 2026-01-31 09:39:15 +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#30192