[PATCH v3 0/1] MR10983: wined3d: Handle data aliasing when blitting to itself with FX transforms.

[Paul Gofman (＠gofman)]

--
  v3: wined3d: Handle data aliasing when blitting to itself with FX transforms.
 
https://gitlab.winehq.org/wine/wine/-/merge_requests/10983

[PATCH v3 1/1] wined3d: Handle data aliasing when blitting to itself with FX transforms.

[Paul Gofman]

From: Paul Gofman <[e-mail]>

---
 dlls/wined3d/surface.c | 21 +++++++++++++++------
 1 file changed, 15 insertions(+), 6 deletions(-)

diff --git a/dlls/wined3d/surface.c b/dlls/wined3d/surface.c
index cf081a17c68..772caa77c35 100644
--- a/dlls/wined3d/surface.c
+++ b/dlls/wined3d/surface.c
@@ -826,9 +826,6 @@ static HRESULT surface_cpu_blt(struct wined3d_texture *dst_texture, unsigned int
                     memcpy(dbuf, sbuf, row_byte_count);
                     dbuf += dst_map.row_pitch;
                 }
-
-                if (same_sub_resource)
-                    free(tmp_buffer);
             }
         }
         else
@@ -904,9 +901,6 @@ do { \
                 dbuf += dst_map.row_pitch;
                 last_sy = sy;
             }
-
-            if (same_sub_resource)
-                free(tmp_buffer);
         }
     }
     else
@@ -964,6 +958,19 @@ do { \
             dBottomLeft  = dTopLeft + ((dst_height - 1) * dst_map.row_pitch);
             dBottomRight = dBottomLeft + ((dst_width - 1) * bpp);
 
+            if (same_sub_resource &&
+                    !(dst_box->bottom <= src_box->top || src_box->bottom <= dst_box->top
+                    || dst_box->right <= src_box->left || src_box->right <= dst_box->left)
+                    && fx->fx & (WINEDDBLTFX_MIRRORLEFTRIGHT | WINEDDBLTFX_MIRRORUPDOWN | WINEDDBLTFX_ROTATE180
+                    | WINEDDBLTFX_ROTATE270 | WINEDDBLTFX_ROTATE90))
+            {
+                if ((tmp_buffer = malloc(src_height * src_map.row_pitch)))
+                {
+                    memcpy(tmp_buffer, sbase, src_height * src_map.row_pitch);
+                    sbase = tmp_buffer;
+                }
+            }
+
             if (fx->fx & WINEDDBLTFX_ARITHSTRETCHY)
             {
                 /* I don't think we need to do anything about this flag. */
@@ -1112,6 +1119,8 @@ error:
         FIXME("    Unsupported flags %#x.\n", flags);
 
 release:
+    free(tmp_buffer);
+
     if (upload && hr == WINED3D_OK)
     {
         struct wined3d_bo_address data;
-- 
GitLab

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

[Paul Gofman (＠gofman)]

Oh yeah, thanks. That was meant to check if there is any overlap between recrtangles.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/10983#note_141245

[Paul Gofman (＠gofman)]

v3:

* fix rectangles overlap check.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/10983#note_141246

[Henri Verbeet (＠hverbeet)]

Looks good to me like this. (Although I wouldn't complain if you were to add some tests. :))

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/10983#note_141251

[PATCH v3 0/1] MR10983: wined3d: Handle data aliasing when blitting to itself with FX transforms. - approved

[Henri Verbeet (＠hverbeet)]

This merge request was approved by Henri Verbeet.
-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/10983

[PATCH v3 0/1] MR10983: wined3d: Handle data aliasing when blitting to itself with FX transforms. - approved

[Elizabeth Figura (＠zfigura)]

This merge request was approved by Elizabeth Figura.
-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/10983