coffee/wepoll
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

port: fix unlikely bug in _ep_port_wait() time-out behavior

Browse Source
This commit is contained in:
Bert Belder 2018-05-01 22:44:32 +02:00
parent 9d61ddfddb
commit de68d70f9c
No known key found for this signature in database
GPG Key ID: 7A77887B2E2ED461
1 changed files with 11 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
src/port.c
View File

@ -225,7 +225,7 @@ int ep_port_wait(ep_port_t* port_info,
/* Dequeue completion packets until either at least one interesting event /* Dequeue completion packets until either at least one interesting event
* has been discovered, or the timeout is reached. * has been discovered, or the timeout is reached.
*/ */
do { for (;;) {
ULONGLONG now; ULONGLONG now;
result = result =
@ -234,16 +234,20 @@ int ep_port_wait(ep_port_t* port_info,
break; /* Result, error, or time-out. */ break; /* Result, error, or time-out. */
if (timeout < 0) if (timeout < 0)
continue; /* _ep_port_wait() never times out. */ continue; /* When timeout is negative, never time out. */
/* Check for time-out. */ /* Update time. */
now = GetTickCount64(); now = GetTickCount64();
if (now >= due)
break;
/* Recompute timeout. */ /* Do not allow the due time to be in the past. */
if (now >= due) {
SetLastError(WAIT_TIMEOUT);
break;
}
/* Recompute time-out argument for GetQueuedCompletionStatus. */
gqcs_timeout = (DWORD)(due - now); gqcs_timeout = (DWORD)(due - now);
} while (gqcs_timeout > 0); }
_ep_port_update_events_if_polling(port_info); _ep_port_update_events_if_polling(port_info);