Marcus Meissner : winhttp: Added missing LeaveCriticalSection (Coverity).

Alexandre Julliard julliard at winehq.org
Thu Nov 24 15:04:38 CST 2011


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

Author: Marcus Meissner <meissner at suse.de>
Date:   Thu Nov 24 02:20:56 2011 +0100

winhttp: Added missing LeaveCriticalSection (Coverity).

---

 dlls/winhttp/request.c |    5 ++++-
 1 files changed, 4 insertions(+), 1 deletions(-)

diff --git a/dlls/winhttp/request.c b/dlls/winhttp/request.c
index 257fc97..a56572d 100644
--- a/dlls/winhttp/request.c
+++ b/dlls/winhttp/request.c
@@ -2973,7 +2973,10 @@ static HRESULT WINAPI winhttp_request_Send(
         return S_OK;
     }
     VariantClear( &request->data );
-    if ((hr = VariantCopyInd( &request->data, &body )) != S_OK) return hr;
+    if ((hr = VariantCopyInd( &request->data, &body )) != S_OK) {
+        LeaveCriticalSection( &request->cs );
+        return hr;
+    }
 
     if (request->wait) /* async request */
     {




More information about the wine-cvs mailing list