[PATCH 0/1] MR6627: gdiplus: Add support for EmfPlusRecordTypeSetRenderingOrigin record playback.

[PATCH 1/1] gdiplus: Add support for EmfPlusRecordTypeSetRenderingOrigin record playback.

[Dmitry Timoshkov]

From: Dmitry Timoshkov

Signed-off-by: Dmitry Timoshkov 
---
 dlls/gdiplus/metafile.c | 10 +++++++++-
 1 file changed, 9 insertions(+), 1 deletion(-)

diff --git a/dlls/gdiplus/metafile.c b/dlls/gdiplus/metafile.c
index c70aa333428..ea72d6bda34 100644
--- a/dlls/gdiplus/metafile.c
+++ b/dlls/gdiplus/metafile.c
@@ -3748,8 +3748,16 @@ GpStatus WINGDIPAPI GdipPlayMetafileRecord(GDIPCONST GpMetafile *metafile,

return stat;
         }
+        case EmfPlusRecordTypeSetRenderingOrigin:
+        {
+            const EmfPlusSetRenderingOrigin *origin = (const EmfPlusSetRenderingOrigin *)header;
+            return GdipSetRenderingOrigin(real_metafile->playback_graphics, origin->x, origin->y);
+        }
         default:
-            FIXME("Not implemented for record type %x\n", recordType);
+            if (recordType >= GDIP_EMFPLUS_RECORD_BASE)
+                FIXME("Not implemented for EMF+ record type %u\n", recordType - GDIP_EMFPLUS_RECORD_BASE);
+            else
+                FIXME("Not implemented for record type %x\n", recordType);
             return NotImplemented;
         }
     }

[Esme Povirk (＠madewokherd)]

Esme Povirk (@madewokherd) commented about dlls/gdiplus/metafile.c:
>          return stat;
>      }
> -
>    case EmfPlusRecordTypeSetRenderingOrigin:
> -
>    {
> -
>        const EmfPlusSetRenderingOrigin *origin = (const EmfPlusSetRenderingOrigin *)header;
> -
>        return GdipSetRenderingOrigin(real_metafile->playback_graphics, origin->x, origin->y);

Please validate the record size before dereferencing these.