Pierre Schweitzer : wineqtdecoder: Fix handle leak.

Alexandre Julliard julliard at winehq.org
Tue Jan 3 12:52:54 CST 2012


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

Author: Pierre Schweitzer <pierre at reactos.org>
Date:   Fri Dec 30 21:55:06 2011 +0100

wineqtdecoder: Fix handle leak.

---

 dlls/wineqtdecoder/qtsplitter.c |   11 +++++++++--
 1 files changed, 9 insertions(+), 2 deletions(-)

diff --git a/dlls/wineqtdecoder/qtsplitter.c b/dlls/wineqtdecoder/qtsplitter.c
index 394fc40..06709c3 100644
--- a/dlls/wineqtdecoder/qtsplitter.c
+++ b/dlls/wineqtdecoder/qtsplitter.c
@@ -892,6 +892,7 @@ static HRESULT QT_Process_Movie(QTSplitter* filter)
     Track trk;
     short id = 0;
     DWORD tid;
+    HANDLE thread;
 
     TRACE("Trying movie connect\n");
 
@@ -930,8 +931,14 @@ static HRESULT QT_Process_Movie(QTSplitter* filter)
     if (trk)
         hr = QT_Process_Audio_Track(filter, trk);
 
-    CreateThread(NULL, 0, QTSplitter_thread, filter, 0, &tid);
-    TRACE("Created thread 0x%08x\n",tid);
+    thread = CreateThread(NULL, 0, QTSplitter_thread, filter, 0, &tid);
+    if (thread)
+    {
+        TRACE("Created thread 0x%08x\n", tid);
+        CloseHandle(thread);
+    }
+    else
+        hr = HRESULT_FROM_WIN32(GetLastError());
 
     return hr;
 }




More information about the wine-cvs mailing list