From 6ed28ada49bdcb8dbd0cb6a6e5f90452865ed68d Mon Sep 17 00:00:00 2001 From: Alexandre Julliard Date: Thu, 3 Nov 2005 11:38:48 +0000 Subject: [PATCH] Fixed fd leaks (spotted by Cihan Altinay). --- dlls/kernel/comm.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/dlls/kernel/comm.c b/dlls/kernel/comm.c index 2f75e2fb9b7..0f2bf172ba1 100644 --- a/dlls/kernel/comm.c +++ b/dlls/kernel/comm.c @@ -2095,7 +2095,6 @@ static BOOL COMM_WaitCommEvent( { TRACE("Event already met\n"); *lpdwEvents = result_mask; - release_comm_fd( commio->handle, fd ); HeapFree(GetProcessHeap(), 0, commio ); res = TRUE; } @@ -2105,10 +2104,12 @@ static BOOL COMM_WaitCommEvent( SetLastError(ERROR_IO_PENDING); res = FALSE; } + release_comm_fd( hFile, fd ); return res; #if !defined(TIOCINQ) || (!(defined(TIOCSERGETLSR) && defined(TIOCSER_TEMT)) || !defined(TIOCINQ)) || !defined(TIOCMGET) || !defined(TIOCM_CTS) ||!defined(TIOCM_DSR) || !defined(TIOCM_RNG) || !defined(TIOCM_CAR) error: FIXME("Returning error because of missing capabilities\n"); + release_comm_fd( hFile, fd ); HeapFree(GetProcessHeap(), 0, commio ); SetLastError(ERROR_INVALID_PARAMETER); return FALSE; -- 2.11.4.GIT