From a4186cffbfdb464ff1b7344fd7561d6014ef721e Mon Sep 17 00:00:00 2001 From: Pino Toscano Date: Thu, 10 May 2012 14:53:21 -0700 Subject: [PATCH] Hurd: mach: compliance fixes for nanosleep --- ChangeLog | 40 +++++++++++++++++++++++----------------- sysdeps/mach/nanosleep.c | 33 ++++++++++++++++++++++++--------- 2 files changed, 47 insertions(+), 26 deletions(-) diff --git a/ChangeLog b/ChangeLog index 456c632..c568aec 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,9 @@ +2012-05-10 Pino Toscano + + * sysdeps/mach/nanosleep.c: Return EINVAL for invalid values of + REQUESTED_TIME. Properly set the remaining time and return EINTR + if interrupted. + 2012-05-10 Thomas Schwinge * sysdeps/mach/hurd/Makefile ($(common-objpfx)linkobj/libc.so): @@ -5,13 +11,13 @@ 2012-05-10 Samuel Thibault - * sysdeps/generic/ldsodefs.h [LIBC_STACK_END_NOT_RELRO] - (__libc_stack_end): Do not use attribute_relro. - * sysdeps/mach/hurd/dl-sysdep.h (LIBC_STACK_END_NOT_RELRO): Define. - * sysdeps/mach/hurd/i386/init-first.c (init): Update __libc_stack_end + * sysdeps/generic/ldsodefs.h [LIBC_STACK_END_NOT_RELRO] + (__libc_stack_end): Do not use attribute_relro. + * sysdeps/mach/hurd/dl-sysdep.h (LIBC_STACK_END_NOT_RELRO): Define. + * sysdeps/mach/hurd/i386/init-first.c (init): Update __libc_stack_end to libthread-provided value. - * sysdeps/mach/hurd/dl-sysdep.c (__libc_stack_end): Do not use - attribute_relro. + * sysdeps/mach/hurd/dl-sysdep.c (__libc_stack_end): Do not use + attribute_relro. 2012-05-10 Thomas Schwinge @@ -36,7 +42,7 @@ 2012-05-10 Samuel Thibault - * sysdeps/mach/hurd/setitimer.c (setitimer_locked): Use common exit + * sysdeps/mach/hurd/setitimer.c (setitimer_locked): Use common exit path instead of returning without unlocking. * sysdeps/mach/hurd/bits/ioctls.h (_IOIW): New macro for @@ -63,16 +69,16 @@ * sysdeps/mach/hurd/dl-sysdep.c: Conditionalize contents on [SHARED]. - * hurd/hurd/fd.h (_hurd_fd_get): Call HURD_CRITICAL_BEGIN/ - HURD_CRITICAL_END around holding _hurd_dtable_lock. - * sysdeps/mach/hurd/dirfd (dirfd): Likewise. - * sysdeps/mach/hurd/opendir.c (_hurd_fd_opendir): Call - HURD_CRITICAL_BEGIN/HURD_CRITICAL_END around holding - d->port.lock. + * hurd/hurd/fd.h (_hurd_fd_get): Call HURD_CRITICAL_BEGIN/ + HURD_CRITICAL_END around holding _hurd_dtable_lock. + * sysdeps/mach/hurd/dirfd (dirfd): Likewise. + * sysdeps/mach/hurd/opendir.c (_hurd_fd_opendir): Call + HURD_CRITICAL_BEGIN/HURD_CRITICAL_END around holding + d->port.lock. - * hurd/catch-signal.c (hurd_catch_signal): Use sigsetjmp/siglongjmp - instead of setjmp/longjmp to restore the signal mask. Call sigsetjmp - when handler == SIG_ERR, not when handler != SIG_ERR. + * hurd/catch-signal.c (hurd_catch_signal): Use sigsetjmp/siglongjmp + instead of setjmp/longjmp to restore the signal mask. Call sigsetjmp + when handler == SIG_ERR, not when handler != SIG_ERR. 2012-05-10 Thomas Schwinge @@ -95,7 +101,7 @@ 2012-05-10 Samuel Thibault - * bits/in.h (SOL_IP, SOL_IPV6, SOL_ICMPV6): New macros. + * bits/in.h (SOL_IP, SOL_IPV6, SOL_ICMPV6): New macros. 2012-05-10 Thomas Schwinge diff --git a/sysdeps/mach/nanosleep.c b/sysdeps/mach/nanosleep.c index 0192ff0..a1ba19c 100644 --- a/sysdeps/mach/nanosleep.c +++ b/sysdeps/mach/nanosleep.c @@ -1,5 +1,5 @@ /* nanosleep -- sleep for a period specified with a struct timespec - Copyright (C) 2002 Free Software Foundation, Inc. + Copyright (C) 2002-2012 Free Software Foundation, Inc. This file is part of the GNU C Library. The GNU C Library is free software; you can redistribute it and/or @@ -27,6 +27,15 @@ __nanosleep (const struct timespec *requested_time, { mach_port_t recv; struct timeval before, after; + + if (requested_time->tv_sec < 0 + || requested_time->tv_nsec < 0 + || requested_time->tv_nsec >= 1000000000) + { + errno = EINVAL; + return -1; + } + const mach_msg_timeout_t ms = requested_time->tv_sec * 1000 + (requested_time->tv_nsec + 999999) / 1000000; @@ -35,16 +44,22 @@ __nanosleep (const struct timespec *requested_time, if (remaining && __gettimeofday (&before, NULL) < 0) return -1; - (void) __mach_msg (NULL, MACH_RCV_MSG|MACH_RCV_TIMEOUT|MACH_RCV_INTERRUPT, - 0, 0, recv, ms, MACH_PORT_NULL); + error_t err = __mach_msg (NULL, MACH_RCV_MSG|MACH_RCV_TIMEOUT|MACH_RCV_INTERRUPT, + 0, 0, recv, ms, MACH_PORT_NULL); __mach_port_destroy (mach_task_self (), recv); - if (remaining && __gettimeofday (&after, NULL) < 0) - return -1; - - if (remaining) + if (err == EMACH_RCV_INTERRUPTED) { - timersub (&after, &before, &after); - TIMEVAL_TO_TIMESPEC (&after, remaining); + if (remaining && __gettimeofday (&after, NULL) >= 0) + { + struct timeval req_time, elapsed, rem; + TIMESPEC_TO_TIMEVAL (&req_time, requested_time); + timersub (&after, &before, &elapsed); + timersub (&req_time, &elapsed, &rem); + TIMEVAL_TO_TIMESPEC (&rem, remaining); + } + + errno = EINTR; + return -1; } return 0; -- 2.7.4