aboutsummaryrefslogtreecommitdiff
path: root/hurd/geteuids.c
diff options
context:
space:
mode:
authorSamuel Thibault <samuel.thibault@ens-lyon.org>2021-03-23 22:38:54 +0000
committerSamuel Thibault <samuel.thibault@ens-lyon.org>2021-03-23 22:40:10 +0000
commitc3b287be74e12bdb15ed36e98d24fd42f8ee9122 (patch)
tree82a9f85685a35bf5b12e480ee6ed1abc24103dc8 /hurd/geteuids.c
parenta4ea18ec6cc3f1370efab09bc18c1badbf74aa13 (diff)
downloadglibc-c3b287be74e12bdb15ed36e98d24fd42f8ee9122.zip
glibc-c3b287be74e12bdb15ed36e98d24fd42f8ee9122.tar.gz
glibc-c3b287be74e12bdb15ed36e98d24fd42f8ee9122.tar.bz2
hurd: handle EINTR during critical sections
During critical sections, signal handling is deferred and thus RPCs return EINTR, even if SA_RESTART is set. We thus have to restart the whole critical section in that case. This also adds HURD_CRITICAL_UNLOCK in the cases where one wants to break the section in the middle.
Diffstat (limited to 'hurd/geteuids.c')
-rw-r--r--hurd/geteuids.c4
1 files changed, 4 insertions, 0 deletions
diff --git a/hurd/geteuids.c b/hurd/geteuids.c
index 840d80a..7fb8b0e 100644
--- a/hurd/geteuids.c
+++ b/hurd/geteuids.c
@@ -26,6 +26,7 @@ geteuids (int n, uid_t *uidset)
int nuids;
void *crit;
+retry:
crit = _hurd_critical_section_lock ();
__mutex_lock (&_hurd_id.lock);
@@ -33,6 +34,9 @@ geteuids (int n, uid_t *uidset)
{
__mutex_unlock (&_hurd_id.lock);
_hurd_critical_section_unlock (crit);
+ if (err == EINTR)
+ /* Got a signal while inside an RPC of the critical section, retry again */
+ goto retry;
return __hurd_fail (err);
}