Re: ole32/tests: guard ReleaseStgMedium with SUCCEEDED
29 Sep
2009
29 Sep
'09
8:29 a.m.
<Joerg-Cyril.Hoehle(a)t-systems.com> writes:
@@ -1401,7 +1405,7 @@ static void test_getdatahere(void) hr = IDataObject_GetDataHere(get, &fmt, &med); ok(hr == S_OK, "got %08x\n", hr); ok(med.tymed == TYMED_HGLOBAL, "got %x\n", med.tymed); - ReleaseStgMedium(&med); + if(SUCCEEDED(hr)) ReleaseStgMedium(&med);
I don't think that's correct for the GetDataHere case, it's allocated by the caller. -- Alexandre Julliard julliard(a)winehq.org
5918
Age (days ago)
5918
Last active (days ago)
0 comments
1 participants
participants (1)
-
Alexandre Julliard