diff --git a/dlls/devenum/devenum_main.c b/dlls/devenum/devenum_main.c index b61e0f8a9ce..4a45971233d 100644 --- a/dlls/devenum/devenum_main.c +++ b/dlls/devenum/devenum_main.c @@ -278,9 +278,7 @@ HRESULT WINAPI DllRegisterServer(void) if (hkeyClsid) RegCloseKey(hkeyClsid); - if (pszClsidDevMon) - CoTaskMemFree(pszClsidDevMon); - + CoTaskMemFree(pszClsidDevMon); CoUninitialize(); return res; diff --git a/dlls/devenum/parsedisplayname.c b/dlls/devenum/parsedisplayname.c index 310ea950a60..091ad08bc2e 100644 --- a/dlls/devenum/parsedisplayname.c +++ b/dlls/devenum/parsedisplayname.c @@ -147,8 +147,7 @@ static HRESULT WINAPI DEVENUM_IParseDisplayName_ParseDisplayName( if (pEm) IEnumMoniker_Release(pEm); - if (pszClass) - CoTaskMemFree(pszClass); + CoTaskMemFree(pszClass); TRACE("-- returning: %lx\n", res); return res; diff --git a/dlls/msdmo/dmort.c b/dlls/msdmo/dmort.c index bf788a5c1cb..d72d376f11a 100644 --- a/dlls/msdmo/dmort.c +++ b/dlls/msdmo/dmort.c @@ -121,11 +121,8 @@ HRESULT WINAPI MoFreeMediaType(DMO_MEDIA_TYPE* pmedia) pmedia->pUnk = NULL; } - if (pmedia->pbFormat) - { - CoTaskMemFree(pmedia->pbFormat); - pmedia->pbFormat = NULL; - } + CoTaskMemFree(pmedia->pbFormat); + pmedia->pbFormat = NULL; return S_OK; } diff --git a/dlls/oleaut32/usrmarshal.c b/dlls/oleaut32/usrmarshal.c index 90782c76752..04b64b92ce3 100644 --- a/dlls/oleaut32/usrmarshal.c +++ b/dlls/oleaut32/usrmarshal.c @@ -1726,8 +1726,7 @@ void CALLBACK ITypeInfo_ReleaseVarDesc_Proxy( { TRACE("(%p, %p)\n", This, pVarDesc); - if(pVarDesc->lpstrSchema) - CoTaskMemFree(pVarDesc->lpstrSchema); + CoTaskMemFree(pVarDesc->lpstrSchema); if(pVarDesc->varkind == VAR_CONST) CoTaskMemFree(pVarDesc->u.lpvarValue); diff --git a/dlls/qcap/enummedia.c b/dlls/qcap/enummedia.c index a57d8a9bfd3..8f3361111a7 100644 --- a/dlls/qcap/enummedia.c +++ b/dlls/qcap/enummedia.c @@ -47,11 +47,9 @@ HRESULT CopyMediaType(AM_MEDIA_TYPE * pDest, const AM_MEDIA_TYPE *pSrc) void FreeMediaType(AM_MEDIA_TYPE * pMediaType) { - if (pMediaType->pbFormat) - { - CoTaskMemFree(pMediaType->pbFormat); - pMediaType->pbFormat = NULL; - } + CoTaskMemFree(pMediaType->pbFormat); + pMediaType->pbFormat = NULL; + if (pMediaType->pUnk) { IUnknown_Release(pMediaType->pUnk); diff --git a/dlls/quartz/filesource.c b/dlls/quartz/filesource.c index a44ca364c56..23fb1fb44dc 100644 --- a/dlls/quartz/filesource.c +++ b/dlls/quartz/filesource.c @@ -620,11 +620,10 @@ static HRESULT WINAPI FileSource_Load(IFileSourceFilter * iface, LPCOLESTR pszFi IPin_Release(This->pOutputPin); This->pOutputPin = NULL; } - if (This->pszFileName) - { - CoTaskMemFree(This->pszFileName); - This->pszFileName = NULL; - } + + CoTaskMemFree(This->pszFileName); + This->pszFileName = NULL; + CloseHandle(hFile); } @@ -1079,12 +1078,8 @@ static HRESULT WINAPI FileAsyncReader_WaitForNext(IAsyncReader * iface, DWORD dw *pdwUser = pDataRq->dwUserData; } - /* clean up */ - if (pDataRq) - { - /* no need to close event handle since we will close it when the pin is destroyed */ - CoTaskMemFree(pDataRq); - } + /* no need to close event handle since we will close it when the pin is destroyed */ + CoTaskMemFree(pDataRq); TRACE("-- %lx\n", hr); return hr; diff --git a/dlls/quartz/filtermapper.c b/dlls/quartz/filtermapper.c index 5a5eaf34fc1..7a92a7e7c09 100644 --- a/dlls/quartz/filtermapper.c +++ b/dlls/quartz/filtermapper.c @@ -152,8 +152,7 @@ static int find_data(struct Vector * v, const BYTE * pData, int size) static void delete_vector(struct Vector * v) { - if (v->pData) - CoTaskMemFree(v->pData); + CoTaskMemFree(v->pData); v->current = 0; v->capacity = 0; } @@ -288,11 +287,8 @@ static HRESULT WINAPI FilterMapper2_CreateCategory( } CloseHandle(hKey); - - if (wClsidCategory) - CoTaskMemFree(wClsidCategory); - if (wClsidAMCat) - CoTaskMemFree(wClsidAMCat); + CoTaskMemFree(wClsidCategory); + CoTaskMemFree(wClsidAMCat); return hr; } @@ -336,10 +332,8 @@ static HRESULT WINAPI FilterMapper2_UnregisterFilter( hr = HRESULT_FROM_WIN32(lRet); } - if (wClsidCategory) - CoTaskMemFree(wClsidCategory); - if (wFilter) - CoTaskMemFree(wFilter); + CoTaskMemFree(wClsidCategory); + CoTaskMemFree(wFilter); return hr; } @@ -368,8 +362,7 @@ static HRESULT FM2_WriteClsid(IPropertyBag * pPropBag, REFCLSID clsid) V_UNION(&var, bstrVal) = wszClsid; hr = IPropertyBag_Write(pPropBag, wszClsidName, &var); } - if (wszClsid) - CoTaskMemFree(wszClsid); + CoTaskMemFree(wszClsid); return hr; } @@ -748,11 +741,9 @@ static HRESULT WINAPI FilterMapper2_RegisterFilter( strcpyW(pCurrent+1, szInstance); else { - if (szClsidTemp) - { - CoTaskMemFree(szClsidTemp); - szClsidTemp = NULL; - } + CoTaskMemFree(szClsidTemp); + szClsidTemp = NULL; + hr = StringFromCLSID(clsidFilter, &szClsidTemp); if (SUCCEEDED(hr)) strcpyW(pCurrent+1, szClsidTemp); @@ -787,16 +778,14 @@ static HRESULT WINAPI FilterMapper2_RegisterFilter( if (pPropBag) IPropertyBag_Release(pPropBag); - if (szClsidTemp) - CoTaskMemFree(szClsidTemp); + CoTaskMemFree(szClsidTemp); if (SUCCEEDED(hr) && ppMoniker) *ppMoniker = pMoniker; else if (pMoniker) IMoniker_Release(pMoniker); - if (pregfp2) - CoTaskMemFree(pregfp2); + CoTaskMemFree(pregfp2); TRACE("-- returning %lx\n", hr); @@ -1360,8 +1349,7 @@ static HRESULT WINAPI FilterMapper_RegisterPin( hr = HRESULT_FROM_WIN32(lRet); } - if (wszClsid) - CoTaskMemFree(wszClsid); + CoTaskMemFree(wszClsid); if (hKey) CloseHandle(hKey); if (hPinsKey) @@ -1437,12 +1425,9 @@ static HRESULT WINAPI FilterMapper_RegisterPinType( CloseHandle(hKey); } - if (wszClsid) - CoTaskMemFree(wszClsid); - if (wszClsidMajorType) - CoTaskMemFree(wszClsidMajorType); - if (wszClsidSubType) - CoTaskMemFree(wszClsidSubType); + CoTaskMemFree(wszClsid); + CoTaskMemFree(wszClsidMajorType); + CoTaskMemFree(wszClsidSubType); return hr; } @@ -1488,8 +1473,7 @@ static HRESULT WINAPI FilterMapper_UnregisterFilter(IFilterMapper * iface, CLSID CloseHandle(hKey); } - if (wszClsid) - CoTaskMemFree(wszClsid); + CoTaskMemFree(wszClsid); return hr; } @@ -1546,8 +1530,7 @@ static HRESULT WINAPI FilterMapper_UnregisterPin(IFilterMapper * iface, CLSID Fi CoTaskMemFree(wszPinNameKey); } - if (wszClsid) - CoTaskMemFree(wszClsid); + CoTaskMemFree(wszClsid); if (hKey) CloseHandle(hKey);