=?UTF-8?Q?Fr=C3=A9d=C3=A9ric=20Delanoy=20?=: windowscodecs: Fix memory deallocation in LoadUnknownMetadata (Coverity).

Alexandre Julliard julliard at winehq.org
Mon Jan 16 13:01:30 CST 2012


Module: wine
Branch: master
Commit: 07b14bf2816e5376fd63aa9fc89023165ad614a7
URL:    http://source.winehq.org/git/wine.git/?a=commit;h=07b14bf2816e5376fd63aa9fc89023165ad614a7

Author: Frédéric Delanoy <frederic.delanoy at gmail.com>
Date:   Sun Jan 15 00:35:47 2012 +0100

windowscodecs: Fix memory deallocation in LoadUnknownMetadata (Coverity).

---

 dlls/windowscodecs/metadatahandler.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/dlls/windowscodecs/metadatahandler.c b/dlls/windowscodecs/metadatahandler.c
index 114c3af..afcff21 100644
--- a/dlls/windowscodecs/metadatahandler.c
+++ b/dlls/windowscodecs/metadatahandler.c
@@ -557,7 +557,7 @@ static HRESULT LoadUnknownMetadata(IStream *input, const GUID *preferred_vendor,
     }
 
     result = HeapAlloc(GetProcessHeap(), 0, sizeof(MetadataItem));
-    if (FAILED(hr))
+    if (!result)
     {
         HeapFree(GetProcessHeap(), 0, data);
         return E_OUTOFMEMORY;




More information about the wine-cvs mailing list