This is clearly fine as a bug fix, of course. One might question if this is the proper place though, i.e. should we offset the dest rect inside `d3dx_load_image_from_memory()`? I think, in terms of API, here is right place. Regardless of what `d3dx_image_decompress()` might need, `d3dx_load_image_from_memory()` receives a plain destination memory pointer with no info about any origin offset (or you can also see this as, `dst_memory` is the address of the top-left pixel, as far as `d3dx_load_image_from_memory()` is concerned). This is effectively a fix for `D3DXLoadSurfaceFromMemory()` passing down bad dest rects. Bottom line, I think the patch is great but the commit subject / message can use some improvement. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/5644#note_70495