Daniel Lehman : oleaut32: Return success from GetSizeMax if not dirty.

[Alexandre Julliard]

Module: wine
Branch: master
Commit: c1f5f98f79b0dce78a19bc6f44e3f0f029fe9668
URL:    https://gitlab.winehq.org/wine/wine/-/commit/c1f5f98f79b0dce78a19bc6f44e3f0f...

Author: Daniel Lehman 
Date:   Wed Mar 13 21:23:02 2024 -0700

oleaut32: Return success from GetSizeMax if not dirty.

---

dlls/oleaut32/olepicture.c       | 3 ++-
 dlls/oleaut32/tests/olepicture.c | 1 -
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/dlls/oleaut32/olepicture.c b/dlls/oleaut32/olepicture.c
index 78fd3fc5681..e4bfafe6fd0 100644
--- a/dlls/oleaut32/olepicture.c
+++ b/dlls/oleaut32/olepicture.c
@@ -1882,7 +1882,8 @@ static HRESULT WINAPI OLEPictureImpl_GetSizeMax(IPersistStream *iface, ULARGE_IN
                 FIXME("(%p), PICTYPE_BITMAP (format UNKNOWN, using BMP?) not implemented!\n",This);
                 break;
             }
-        }
+        } else hr = S_OK;
+
         break;
     case PICTYPE_METAFILE:
         FIXME("(%p), PICTYPE_METAFILE not implemented!\n",This);
diff --git a/dlls/oleaut32/tests/olepicture.c b/dlls/oleaut32/tests/olepicture.c
index 66cc41646c5..07d9ba98f1b 100644
--- a/dlls/oleaut32/tests/olepicture.c
+++ b/dlls/oleaut32/tests/olepicture.c
@@ -1262,7 +1262,6 @@ static void test_load_save_bmp(void)

maxsize.QuadPart = 0;
     hr = IPersistStream_GetSizeMax(src_stream, &maxsize);
-    todo_wine
     ok(hr == S_OK, "GetSizeMax error %#lx\n", hr);
     ok(maxsize.QuadPart == 74, "expected 74, got %s\n", wine_dbgstr_longlong(maxsize.QuadPart));