[PATCH] check return code of ICreateDevEnum_CreateClassEnumerator
Marcus Meissner
marcus at jet.franken.de
Fri May 11 14:29:15 CDT 2007
Hi,
Coverity thinks that pEnumCat might be used uninitialized.
I agree.
Ciao, Marcus
---
dlls/quartz/filtermapper.c | 3 +++
1 files changed, 3 insertions(+), 0 deletions(-)
diff --git a/dlls/quartz/filtermapper.c b/dlls/quartz/filtermapper.c
index 6adce61..841b130 100644
--- a/dlls/quartz/filtermapper.c
+++ b/dlls/quartz/filtermapper.c
@@ -966,6 +966,9 @@ static HRESULT WINAPI FilterMapper2_Enum
if (SUCCEEDED(hr))
hr = ICreateDevEnum_CreateClassEnumerator(pCreateDevEnum, &CLSID_ActiveMovieCategories, &pEnumCat, 0);
+ if (!SUCCEEDED(hr))
+ return hr;
+
while (IEnumMoniker_Next(pEnumCat, 1, &pMonikerCat, NULL) == S_OK)
{
IPropertyBag * pPropBagCat = NULL;
--
1.4.3.4
More information about the wine-patches
mailing list