Skip to content

Commit

Permalink
Cygwin: timer_delete: Fix return value
Browse files Browse the repository at this point in the history
timer_delete() always returned failure.  This issue has been
detected by 'stress-ng --hrtimers 1'.

Fixes: 229ea3f ("Cygwin: posix timers: reimplement using OS timer")
Signed-off-by: Christian Franke <[email protected]>
  • Loading branch information
chrfranke authored and github-cygwin committed Oct 23, 2024
1 parent bdd06f8 commit 6af8fea
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 0 deletions.
1 change: 1 addition & 0 deletions winsup/cygwin/posix_timer.cc
Original file line number Diff line number Diff line change
Expand Up @@ -530,6 +530,7 @@ timer_delete (timer_t timerid)
__leave;
}
delete in_tt;
ret = 0;
}
__except (EFAULT) {}
__endtry
Expand Down
2 changes: 2 additions & 0 deletions winsup/cygwin/release/3.5.5
Original file line number Diff line number Diff line change
Expand Up @@ -10,3 +10,5 @@ Fixes:

- Fix pread() and pwrite() EBADF error after fork().
Addresses: https://sourceware.org/pipermail/cygwin/2024-September/256468.html

- Fix timer_delete() return value which always indicated failure.

0 comments on commit 6af8fea

Please sign in to comment.