Fixed fd leaks (spotted by Cihan Altinay).

This commit is contained in:
Alexandre Julliard 2005-11-03 11:38:48 +00:00
parent 5b22e4d6ef
commit 6ed28ada49
1 changed files with 2 additions and 1 deletions

View File

@ -2095,7 +2095,6 @@ static BOOL COMM_WaitCommEvent(
{ {
TRACE("Event already met\n"); TRACE("Event already met\n");
*lpdwEvents = result_mask; *lpdwEvents = result_mask;
release_comm_fd( commio->handle, fd );
HeapFree(GetProcessHeap(), 0, commio ); HeapFree(GetProcessHeap(), 0, commio );
res = TRUE; res = TRUE;
} }
@ -2105,10 +2104,12 @@ static BOOL COMM_WaitCommEvent(
SetLastError(ERROR_IO_PENDING); SetLastError(ERROR_IO_PENDING);
res = FALSE; res = FALSE;
} }
release_comm_fd( hFile, fd );
return res; 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) #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: error:
FIXME("Returning error because of missing capabilities\n"); FIXME("Returning error because of missing capabilities\n");
release_comm_fd( hFile, fd );
HeapFree(GetProcessHeap(), 0, commio ); HeapFree(GetProcessHeap(), 0, commio );
SetLastError(ERROR_INVALID_PARAMETER); SetLastError(ERROR_INVALID_PARAMETER);
return FALSE; return FALSE;