[ros-diffs] [tfaber] 63250: [ICMP] - Fix socket leak. Spotted by Samuel & cppcheck Note: Wine has this in iphlpapi these days. Should probably sync. CORE-8208 #resolve

Author: tfaber
Date: Mon May 12 06:44:01 2014
New Revision: 63250

URL: http://svn.reactos.org/svn/reactos?rev=63250&view=rev
Log:
[ICMP]
- Fix socket leak. Spotted by Samuel & cppcheck
Note: Wine has this in iphlpapi these days. Should probably sync.
CORE-8208 #resolve

Modified:
    trunk/reactos/dll/win32/icmp/icmp_main.c

Modified: trunk/reactos/dll/win32/icmp/icmp_main.c
URL:
http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/icmp/icmp_main.c…
==============================================================================
--- trunk/reactos/dll/win32/icmp/icmp_main.c	[iso-8859-1] (original)
+++ trunk/reactos/dll/win32/icmp/icmp_main.c	[iso-8859-1] Mon May 12 06:44:01 2014
@@ -179,6 +179,7 @@
 
     icp=HeapAlloc(GetProcessHeap(), 0, sizeof(*icp));
     if (icp==NULL) {
+        closesocket(sid);
         SetLastError(IP_NO_RESOURCES);
         return INVALID_HANDLE_VALUE;
     }