[PATCH v2 0/1] MR7622: comctl32: Avoid segfault in PROPSHEET_DoCommand when psInfo is NULL.

[Yuri Hérouard (＠yurih)]

Fixes https://bugs.winehq.org/show_bug.cgi?id=57987.

In the case of that issue, it does not crash anymore, and the settings are now correctly saved.

--
  v2: comctl32: Avoid segfault in PROPSHEET_DoCommand when psInfo is NULL.

https://gitlab.winehq.org/wine/wine/-/merge_requests/7622

[PATCH v2 1/1] comctl32: Avoid segfault in PROPSHEET_DoCommand when psInfo is NULL.

[Yuri Hérouard]

From: Yuri Hérouard

---
 dlls/comctl32/propsheet.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/dlls/comctl32/propsheet.c b/dlls/comctl32/propsheet.c
index 44b24a56a07..d397f85870d 100644
--- a/dlls/comctl32/propsheet.c
+++ b/dlls/comctl32/propsheet.c
@@ -3326,6 +3326,8 @@ static BOOL PROPSHEET_DoCommand(HWND hwnd, WORD wID)
    	{
                     PropSheetInfo* psInfo = GetPropW(hwnd, PropSheetInfoStr);

+                    if (psInfo == NULL) break;
+
                     /* don't overwrite ID_PSRESTARTWINDOWS or ID_PSREBOOTSYSTEM */
                     if (psInfo->result == 0)
                         psInfo->result = IDOK;

[Yuri Hérouard (＠yurih)]

On Sun Mar 23 01:49:47 2025 +0000, Nikolay Sivov wrote:
> I think it's a reasonable change. One remaining issue though - please
> force push with your full name in the patch author field.

Done.

[PATCH v2 0/1] MR7622: comctl32: Avoid segfault in PROPSHEET_DoCommand when psInfo is NULL. - approved

[Nikolay Sivov (＠nsivov)]

This merge request was approved by Nikolay Sivov.

[PATCH v2 0/1] MR7622: comctl32: Avoid segfault in PROPSHEET_DoCommand when psInfo is NULL. - approved

[Zhiyi Zhang (＠zhiyi)]

This merge request was approved by Zhiyi Zhang.