[PATCH] quartz: removed useless NULL check (Coverity)

Marcus Meissner marcus at jet.franken.de
Sun Jun 26 09:36:03 CDT 2011


Hi,

in all paths leading to it, pOutStream was already dereferenced, so
NULL check is not useful. CID 2017.

Ciao, Marcus
---
 dlls/quartz/acmwrapper.c |    3 +--
 1 files changed, 1 insertions(+), 2 deletions(-)

diff --git a/dlls/quartz/acmwrapper.c b/dlls/quartz/acmwrapper.c
index 7cdaba6..814bf84 100644
--- a/dlls/quartz/acmwrapper.c
+++ b/dlls/quartz/acmwrapper.c
@@ -230,8 +230,7 @@ error:
         ash.pbSrc += ash.cbSrcLengthUsed;
         ash.cbSrcLength -= ash.cbSrcLengthUsed;
 
-        if (pOutSample)
-            IMediaSample_Release(pOutSample);
+        IMediaSample_Release(pOutSample);
         pOutSample = NULL;
 
     }
-- 
1.7.3.4




More information about the wine-patches mailing list