Zhiyi Zhang : gdi32/tests: Test deleting the bitmap used for pattern brush creation.

[Alexandre Julliard]

Module: wine
Branch: master
Commit: 1febcf692f820880b85cbc23c5e1b2370cc4e390
URL:    https://source.winehq.org/git/wine.git/?a=commit;h=1febcf692f820880b85cbc23c...

Author: Zhiyi Zhang 
Date:   Tue Dec  7 17:12:16 2021 +0800

gdi32/tests: Test deleting the bitmap used for pattern brush creation.

Test that the bitmap for pattern brush creation can be deleted after CreatePatternBrush() and the
created brush will still function normally. However, the bitmap object can not be retrieved from
LOGBRUSH.lbHatch if the bitmap is deleted.

Signed-off-by: Zhiyi Zhang 
Signed-off-by: Huw Davies 
Signed-off-by: Alexandre Julliard

---

dlls/gdi32/tests/brush.c | 67 ++++++++++++++++++++++++++++++++++++++++++++++--
 1 file changed, 65 insertions(+), 2 deletions(-)

diff --git a/dlls/gdi32/tests/brush.c b/dlls/gdi32/tests/brush.c
index f04ee22ab6d..0841eb2f2e1 100644
--- a/dlls/gdi32/tests/brush.c
+++ b/dlls/gdi32/tests/brush.c
@@ -120,9 +120,14 @@ static void test_pattern_brush(void)
 {
     char buffer[sizeof(BITMAPINFOHEADER) + 2 * sizeof(RGBQUAD) + 32 * 32 / 8];
     BITMAPINFO *info = (BITMAPINFO *)buffer;
-    HBRUSH brush;
-    HBITMAP bitmap;
+    HBITMAP bitmap, bitmap2, old_bitmap;
+    HBRUSH brush, brush2;
+    HDC hdc, hdc_screen;
+    COLORREF color;
+    BOOL result;
     LOGBRUSH br;
+    BITMAP bm;
+    RECT rect;
     INT ret;
     void *bits;
     DIBSECTION dib;
@@ -269,6 +274,64 @@ static void test_pattern_brush(void)
     ok( !brush, "CreatePatternBrush succeeded\n" );

GlobalFree( mem );
+
+    /* Test deleting bitmap after brush creation */
+    /* Create hdc and bitmaps */
+    hdc_screen = GetDC( NULL );
+    hdc = CreateCompatibleDC( hdc_screen );
+    bitmap = CreateCompatibleBitmap( hdc_screen, 16, 16 );
+    bitmap2 = CreateCompatibleBitmap( hdc_screen, 16, 16 );
+
+    /* Fill the first bitmap with 0xff5511 */
+    old_bitmap = SelectObject( hdc, bitmap );
+    SetRect( &rect, 0, 0, 16, 16 );
+    brush = CreateSolidBrush( 0xff5511 );
+    result = FillRect( hdc, &rect, brush );
+    ok( result, "FillRect failed, error %d.\n", GetLastError() );
+    DeleteObject( brush );
+    color = GetPixel( hdc, 10, 10 );
+    ok( color == 0xff5511, "Expected color %#x, got %#x.\n", 0xff5511, color );
+
+    /* Create a pattern brush with the first bitmap filled with 0xff5511 */
+    brush = CreatePatternBrush( bitmap );
+    ok( brush != NULL, "CreatePatternBrush failed, error %u.\n", GetLastError() );
+
+    /* Delete the first bitmap used for pattern brush creation */
+    SelectObject( hdc, bitmap2 );
+    DeleteObject( bitmap );
+
+    memset( &br, 0, sizeof(br) );
+    ret = GetObjectW( brush, sizeof(br), &br );
+    ok( ret == sizeof(br), "wrong size %u\n", ret );
+    ok( br.lbColor == 0, "wrong color %u\n", br.lbColor );
+    ok( br.lbStyle == BS_PATTERN, "wrong style %u\n", br.lbStyle );
+    ok( (HBITMAP)br.lbHatch == bitmap, "wrong handle %p/%p\n", (HBITMAP)br.lbHatch, bitmap );
+
+    /* The first bitmap is now invalid */
+    memset( &bm, 0, sizeof (bm));
+    ret = GetObjectW( bitmap, sizeof(bm), &bm );
+    ok( !ret, "wrong size %u\n", ret );
+
+    /* Fill hdc with 0xabcdef */
+    brush2 = CreateSolidBrush( 0xabcdef );
+    result = FillRect( hdc, &rect, brush2 );
+    ok( result, "FillRect failed, error %d.\n", GetLastError() );
+    color = GetPixel( hdc, 10, 10 );
+    ok( color == 0xabcdef, "Expected color %#x, got %#x.\n", 0xabcdef, color );
+    DeleteObject( brush2 );
+
+    /* Fill hdc with the brush created with the deleted bitmap */
+    /* FillRect() succeeds and hdc is filled with the deleted bitmap content */
+    result = FillRect( hdc, &rect, brush );
+    ok( result, "FillRect failed, error %d.\n", GetLastError() );
+    color = GetPixel( hdc, 10, 10 );
+    ok( color == 0xff5511, "Expected color %#x, got %#x.\n", 0xff5511, color );
+    DeleteObject( brush );
+
+    SelectObject( hdc, old_bitmap );
+    DeleteObject( bitmap2 );
+    DeleteDC( hdc );
+    ReleaseDC( NULL, hdc_screen );
 }

static void test_palette_brush(void)