[reactos] 02/04: [COMCTL32] Add NULL checks to SetWindowSubclass.

[Thomas Faber]

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=3445129dd314c19b602753...

commit 3445129dd314c19b6027532019e0e703f0b77c3a
Author:     Thomas Faber [e-mail]
AuthorDate: Sun Feb 3 16:41:29 2019 +0100
Commit:     Thomas Faber [e-mail]
CommitDate: Sun Feb 17 09:51:16 2019 +0100

[COMCTL32] Add NULL checks to SetWindowSubclass.

Imports Wine commit 04847e68f87 by Sven Baars.

Fixes crash in comctl32_winetest:subclass.
---
 dll/win32/comctl32/commctrl.c | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/dll/win32/comctl32/commctrl.c b/dll/win32/comctl32/commctrl.c
index d4cb73ae76..a3bbd5fc31 100644
--- a/dll/win32/comctl32/commctrl.c
+++ b/dll/win32/comctl32/commctrl.c
@@ -1266,6 +1266,9 @@ BOOL WINAPI SetWindowSubclass (HWND hWnd, SUBCLASSPROC pfnSubclass,

TRACE ("(%p, %p, %lx, %lx)\n", hWnd, pfnSubclass, uIDSubclass, dwRef);

+    if (!hWnd || !pfnSubclass)
+        return FALSE;
+
    /* Since the window procedure that we set here has two additional arguments,
     * we can't simply set it as the new window procedure of the window. So we
     * set our own window procedure and then calculate the other two arguments