https://git.reactos.org/?p=reactos.git;a=commitdiff;h=b57be0a746fd9b42fed0e…
commit b57be0a746fd9b42fed0e9ac5cf88eaf222ce11d
Author: Victor Martinez Calvo <vicmarcal(a)gmail.com>
AuthorDate: Tue May 3 16:00:21 2022 +0200
Commit: GitHub <noreply(a)github.com>
CommitDate: Tue May 3 16:00:21 2022 +0200
[GDI32] Fix GetObjectW() GDI_OBJECT_TYPE_BRUSH handling. (#4394)
This fixes the last GetObjectW apitest case:
ok_long(GetObjectA(hBrush, 0, &TestStruct), 0);
- When "cbSize" is "0", GetObjectW shouldn't return
"sizeof(LOGBRUSH)" but 0.
- Rely in BRUSH::cjGetObject behavior which returns 0 when cjSize is 0.
Co-authored-by: Víctor Martínez Calvo <victormartinez(a)alub.io>
---
win32ss/gdi/gdi32/objects/gdiobj.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/win32ss/gdi/gdi32/objects/gdiobj.c b/win32ss/gdi/gdi32/objects/gdiobj.c
index 96617881fc4..5f8d8e1f6c3 100644
--- a/win32ss/gdi/gdi32/objects/gdiobj.c
+++ b/win32ss/gdi/gdi32/objects/gdiobj.c
@@ -206,7 +206,7 @@ GetObjectW(
break;
case GDI_OBJECT_TYPE_BRUSH:
- if (!lpBuffer || !cbSize) return sizeof(LOGBRUSH);
+ if (!lpBuffer) return sizeof(LOGBRUSH);
break;
case GDI_OBJECT_TYPE_BITMAP: