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

[YuriK7 (＠riyu12383)]

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

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

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

[YuriK7]

From: YuriK7 <riyu12383(a)gmail.com>

---
 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;
-- 
GitLab

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

[Nikolay Sivov (＠nsivov)]

I comment on the bug report, but also mentioning here. It would be interesting to know why it's NULL.

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

[Nikolay Sivov (＠nsivov)]

I think it's a reasonable change. One remaining issue though - please force push with your full name in the patch author field, currently it says YuriK7, which I'm assuming is not the full name.

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