ole32: Fix error handling in GetTimeOfLastChange() for item moniker (PVS-Studio).

f294ea83 · Nikolay Sivov · Alexandre Julliard · 7aa2e9a9 · f294ea83
Commit f294ea83 authored Mar 20, 2015 by Nikolay Sivov Committed by Alexandre Julliard Mar 20, 2015
Hide whitespace changes
Inline Side-by-side

Showing with 6 additions and 0 deletions

itemmoniker.c dlls/ole32/itemmoniker.c +6 -0

No files found.
--- a/dlls/ole32/itemmoniker.c
+++ b/dlls/ole32/itemmoniker.c
@@ -601,8 +601,14 @@ static HRESULT WINAPI ItemMonikerImpl_GetTimeOfLastChange(IMoniker* iface,
        /* IMoniker::GetTimeOfLastChange on the pmkToLeft parameter.                                            */

        res=CreateGenericComposite(pmkToLeft,iface,&compositeMk);
+        if (FAILED(res))
+            return res;

        res=IBindCtx_GetRunningObjectTable(pbc,&rot);
+        if (FAILED(res)) {
+            IMoniker_Release(compositeMk);
+            return res;
+        }

        if (IRunningObjectTable_GetTimeOfLastChange(rot,compositeMk,pItemTime)!=S_OK)