diff options
author | Rich Felker <dalias@aerifal.cx> | 2011-09-04 21:37:07 -0400 |
---|---|---|
committer | Rich Felker <dalias@aerifal.cx> | 2011-09-04 21:37:07 -0400 |
commit | a7778dae226fbae335383bc92b6cdfccc5ea9f2e (patch) | |
tree | 3a290f99c9f397901a3011d04ba8e87907568e44 /src | |
parent | 22e4542348352235614b57948025bdd0cc30dc32 (diff) | |
download | musl-a7778dae226fbae335383bc92b6cdfccc5ea9f2e.tar.gz |
handle pending cancellation when enabling async cancellation
this is not strictly required by the standard, but without it, there
is a race condition where cancellation arriving just before async
cancellation is enabled might not be acted upon. it is impossible for
a conforming application to work around this race condition since
calling pthread_testcancel after setting async cancellation mode is
not allowed (pthread_testcancel is not specified to be
async-cancel-safe). thus the implementation should be responsible for
eliminating the race, from a quality-of-implementation standpoint.
Diffstat (limited to 'src')
-rw-r--r-- | src/thread/pthread_setcanceltype.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/src/thread/pthread_setcanceltype.c b/src/thread/pthread_setcanceltype.c index 7eb543a8..ce2fff07 100644 --- a/src/thread/pthread_setcanceltype.c +++ b/src/thread/pthread_setcanceltype.c @@ -6,5 +6,6 @@ int pthread_setcanceltype(int new, int *old) if (new > 1U) return EINVAL; if (old) *old = self->cancelasync; self->cancelasync = new; + if (new) pthread_testcancel(); return 0; } |