wininet: Fix a couple of reversed checks that could result in leaked socket handles.

Francois Gouget fgouget at codeweavers.com
Thu Mar 8 08:30:41 CST 2007


---
 dlls/wininet/ftp.c |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/dlls/wininet/ftp.c b/dlls/wininet/ftp.c
index 1ebea43..c00b6fd 100644
--- a/dlls/wininet/ftp.c
+++ b/dlls/wininet/ftp.c
@@ -2007,7 +2007,7 @@ HINTERNET FTP_Connect(LPWININETAPPINFOW hIC, LPCWSTR lpszServerName,
     }
 
 lerror:
-    if (!bSuccess && nsocket == -1)
+    if (!bSuccess && nsocket != -1)
         closesocket(nsocket);
 
     if (!bSuccess && lpwfs)
@@ -2378,7 +2378,7 @@ static BOOL FTP_InitListenSocket(LPWININETFTPSESSIONW lpwfs)
         bSuccess = TRUE;
 
 lend:
-    if (!bSuccess && lpwfs->lstnSocket == -1)
+    if (!bSuccess && lpwfs->lstnSocket != -1)
     {
         closesocket(lpwfs->lstnSocket);
         lpwfs->lstnSocket = -1;
-- 
1.4.4.4




More information about the wine-patches mailing list