gdiplus: In GdipImageSelectActiveFrame rely on codec->select_func() to fail.

Signed-off-by: Dmitry Timoshkov <dmitry@baikal.ru>

gdiplus: In GdipImageSelectActiveFrame rely on codec->select_func() to fail.
e7b4e883 · Dmitry Timoshkov · Alexandre Julliard · 408972cb · e7b4e883 · e7b4e883
Commit e7b4e883 authored Mar 23, 2023 by Dmitry Timoshkov Committed by Alexandre Julliard Sep 06, 2023
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 5 deletions

image.c dlls/gdiplus/image.c +1 -4

image.c dlls/gdiplus/tests/image.c +1 -1

No files found.
--- a/dlls/gdiplus/image.c
+++ b/dlls/gdiplus/image.c
@@ -4553,11 +4553,8 @@ GpStatus WINGDIPAPI GdipImageSelectActiveFrame(GpImage *image, GDIPCONST GUID *d
        return ObjectBusy;
    if (frame >= image->frame_count)
-    {
        WARN("requested frame %u, but image has only %u\n", frame, image->frame_count);
-        image_unlock(image);
+        /* rely on codec->select_func() to fail */
-        return InvalidParameter;
-    }
    if (image->type != ImageTypeBitmap && image->type != ImageTypeMetafile)
    {

--- a/dlls/gdiplus/tests/image.c
+++ b/dlls/gdiplus/tests/image.c
@@ -471,7 +471,7 @@ static void test_GdipImageGetFrameDimensionsCount(void)
    expect(0xffffffff, color);
    stat = GdipImageSelectActiveFrame((GpImage*)bm, &dimension, 1);
-    todo_wine expect(Ok, stat);
+    expect(Ok, stat);
    GdipDisposeImage((GpImage*)bm);
 }