[MSAFD]
authorCameron Gutman <aicommander@gmail.com>
Sat, 18 Aug 2012 16:36:05 +0000 (16:36 +0000)
committerCameron Gutman <aicommander@gmail.com>
Sat, 18 Aug 2012 16:36:05 +0000 (16:36 +0000)
- Fix a memory leak in the failure path of WSPGetSockName and WSPGetPeerName
- Fix a broken debug print in WSPGetPeerName

svn path=/trunk/; revision=57097

reactos/dll/win32/msafd/misc/dllmain.c

index f582f4c..15f91af 100644 (file)
@@ -1842,6 +1842,8 @@ WSPGetSockName(IN SOCKET Handle,
         }
     }
 
+    HeapFree(GlobalHeap, 0, TdiAddress);
+
     return MsafdReturnWithErrno ( Status, lpErrno, 0, NULL );
 }
 
@@ -1933,7 +1935,7 @@ WSPGetPeerName(IN SOCKET s,
                            SocketAddress->Address[0].Address,
                            SocketAddress->Address[0].AddressLength);
             *NameLength = Socket->SharedData.SizeOfRemoteAddress;
-            AFD_DbgPrint (MID_TRACE, ("NameLength %d Address: %s Port %x\n",
+            AFD_DbgPrint (MID_TRACE, ("NameLength %d Address: %x Port %x\n",
                           *NameLength, ((struct sockaddr_in *)Name)->sin_addr.s_addr,
                           ((struct sockaddr_in *)Name)->sin_port));
             HeapFree(GlobalHeap, 0, SocketAddress);
@@ -1947,6 +1949,8 @@ WSPGetPeerName(IN SOCKET s,
         }
     }
 
+    HeapFree(GlobalHeap, 0, SocketAddress);
+
     return MsafdReturnWithErrno ( Status, lpErrno, 0, NULL );
 }