oleaut32: Implicitly initialize COM in OleLoadPicturePath.

diff --git a/dlls/oleaut32/olepicture.c b/dlls/oleaut32/olepicture.c
index 07d4dde8f2e0ff8ab10b59ae5887efafa9d43b31..b8e949d2bd8c83ea480317df3a6d93ca6fc825e2 100644 (file)
--- a/dlls/oleaut32/olepicture.c
+++ b/dlls/oleaut32/olepicture.c
@@ -2273,6 +2273,7 @@ HRESULT WINAPI OleLoadPicturePath( LPOLESTR szURLorPath, LPUNKNOWN punkCaller,
   BOOL bRead;
   IPersistStream *pStream;
   HRESULT hRes;
+  HRESULT init_res;
   WCHAR *file_candidate;
   WCHAR path_buf[MAX_PATH];
 
@@ -2349,34 +2350,32 @@ HRESULT WINAPI OleLoadPicturePath( LPOLESTR szURLorPath, LPUNKNOWN punkCaller,
          return hRes;
   }
 
-  hRes = CoCreateInstance(&CLSID_StdPicture, punkCaller, CLSCTX_INPROC_SERVER, 
-                  &IID_IPicture, (LPVOID*)&ipicture);
-  if (hRes != S_OK) {
-      IStream_Release(stream);
-      return hRes;
-  }
-  
-  hRes = IPicture_QueryInterface(ipicture, &IID_IPersistStream, (LPVOID*)&pStream);
-  if (hRes) {
-      IStream_Release(stream);
+  init_res = CoInitialize(NULL);
+
+  hRes = CoCreateInstance(&CLSID_StdPicture, punkCaller, CLSCTX_INPROC_SERVER,
+                          &IID_IPicture, (LPVOID*)&ipicture);
+  if (SUCCEEDED(hRes)) {
+      hRes = IPicture_QueryInterface(ipicture, &IID_IPersistStream, (LPVOID*)&pStream);
+
+      if (SUCCEEDED(hRes)) {
+          hRes = IPersistStream_Load(pStream, stream);
+
+          if (SUCCEEDED(hRes)) {
+              hRes = IPicture_QueryInterface(ipicture, riid, ppvRet);
+
+              if (FAILED(hRes))
+                  ERR("Failed to get interface %s from IPicture.\n", debugstr_guid(riid));
+          }
+          IPersistStream_Release(pStream);
+      }
       IPicture_Release(ipicture);
-      return hRes;
   }
 
-  hRes = IPersistStream_Load(pStream, stream); 
-  IPersistStream_Release(pStream);
   IStream_Release(stream);
 
-  if (hRes) {
-      IPicture_Release(ipicture);
-      return hRes;
-  }
+  if (SUCCEEDED(init_res))
+      CoUninitialize();
 
-  hRes = IPicture_QueryInterface(ipicture,riid,ppvRet);
-  if (hRes)
-      ERR("Failed to get interface %s from IPicture.\n",debugstr_guid(riid));
-  
-  IPicture_Release(ipicture);
   return hRes;
 }
 

diff --git a/dlls/oleaut32/tests/olepicture.c b/dlls/oleaut32/tests/olepicture.c
index 6bf3c445263e9aac6b03b5935d068962e098a8ca..8ea1a2aff65993e25d270ca2a57048f0f5c40d1d 100644 (file)
--- a/dlls/oleaut32/tests/olepicture.c
+++ b/dlls/oleaut32/tests/olepicture.c
@@ -813,7 +813,6 @@ static void test_OleLoadPicturePath(void)
 
     /* Try a normal DOS path. */
     hres = OleLoadPicturePath(temp_fileW + 8, NULL, 0, 0, &IID_IPicture, (void **)&pic);
-    todo_wine
     ok(hres == S_OK ||
        broken(hres == E_UNEXPECTED), /* NT4/Win95 */
        "Expected OleLoadPicturePath to return S_OK, got 0x%08x\n", hres);
@@ -822,7 +821,6 @@ static void test_OleLoadPicturePath(void)
 
     /* Try a DOS path with tacked on "file:". */
     hres = OleLoadPicturePath(temp_fileW, NULL, 0, 0, &IID_IPicture, (void **)&pic);
-    todo_wine
     ok(hres == S_OK ||
        broken(hres == E_UNEXPECTED), /* NT4/Win95 */
        "Expected OleLoadPicturePath to return S_OK, got 0x%08x\n", hres);
@@ -859,7 +857,6 @@ static void test_OleLoadPicturePath(void)
     }
 
     hres = OleLoadPicturePath(temp_fileW, NULL, 0, 0, &IID_IPicture, (void **)&pic);
-    todo_wine
     ok(hres == S_OK ||
        broken(hres == E_UNEXPECTED), /* NT4/Win95 */
        "Expected OleLoadPicturePath to return S_OK, got 0x%08x\n", hres);