Re: [PATCH 0/1] MR8988: win32u: change the stretch mode of dst hdc from BlackOnWhite to ColorOnColor...

[Jinoh Kang (＠iamahuman)]

Jinoh Kang (@iamahuman) commented about dlls/win32u/bitblt.c:
>  }
>  else bmpWork = NtGdiCreateCompatibleBitmap( hdcDest, widthDest, heightDest );
>  oldWork = NtGdiSelectBitmap(hdcWork, bmpWork);
> -
> - if (!(dc_work = get_dc_ptr(hdcWork))) goto error;
> - dc_work->attr->stretch_blt_mode = COLORONCOLOR;

This causes memory leak because get_dc_ptr increments reference count but you don't decrement it back. Call release_dc_ptr when you're done:

```suggestion:-1+0
    if (!(dc_work = get_dc_ptr(hdcWork))) goto error;
    dc_work->attr->stretch_blt_mode = COLORONCOLOR;
    release_dc_ptr( dc_work );
```