From 233ba85cfe54c7149a5c331ae9de8ba34681dcb4 Mon Sep 17 00:00:00 2001 From: Thomas Faber Date: Mon, 12 May 2014 06:44:01 +0000 Subject: [PATCH] [ICMP] - Fix socket leak. Spotted by Samuel & cppcheck Note: Wine has this in iphlpapi these days. Should probably sync. CORE-8208 #resolve svn path=/trunk/; revision=63250 --- reactos/dll/win32/icmp/icmp_main.c | 1 + 1 file changed, 1 insertion(+) diff --git a/reactos/dll/win32/icmp/icmp_main.c b/reactos/dll/win32/icmp/icmp_main.c index 8257c1156b1..187b310f0a6 100644 --- a/reactos/dll/win32/icmp/icmp_main.c +++ b/reactos/dll/win32/icmp/icmp_main.c @@ -179,6 +179,7 @@ HANDLE WINAPI IcmpCreateFile(VOID) icp=HeapAlloc(GetProcessHeap(), 0, sizeof(*icp)); if (icp==NULL) { + closesocket(sid); SetLastError(IP_NO_RESOURCES); return INVALID_HANDLE_VALUE; }