diff options
author | Lucas A. M. Magalhaes <lamm@linux.ibm.com> | 2020-12-01 18:05:07 -0300 |
---|---|---|
committer | Tulio Magno Quites Machado Filho <tuliom@linux.ibm.com> | 2020-12-01 18:26:26 -0300 |
commit | 61855081017dff30c577855cda882740356b5d98 (patch) | |
tree | 69472c92f3f8b65c240b5987bbeee2760108933b /csu | |
parent | 33fc34521de970153344cfe1bfa9ce6da7a6efea (diff) | |
download | glibc-61855081017dff30c577855cda882740356b5d98.zip glibc-61855081017dff30c577855cda882740356b5d98.tar.gz glibc-61855081017dff30c577855cda882740356b5d98.tar.bz2 |
nptl: Fix __futex_clocklock64 return error check [BZ #26964]
The earlier implementation of this, __lll_clocklock, calls lll_clockwait
that doesn't return the futex syscall error codes. It always tries again
if that fails.
However in the current implementation, when the futex returns EAGAIN,
__futex_clocklock64 will also return EGAIN, even if the futex is taken.
This patch fixes the EAGAIN issue and also adds a check for EINTR. As
futex syscall can return EINTR if the thread is interrupted by a signal.
In this case I'm assuming the function should continue trying to lock as
there is no mention to about it on POSIX. Also add a test for both
scenarios.
Reviewed-by: Adhemerval Zanella <adhemerval.zanella@linaro.org>
Diffstat (limited to 'csu')
0 files changed, 0 insertions, 0 deletions