diff options
| author | Joel Sherrill <joel.sherrill@oarcorp.com> | 2013-12-09 15:36:38 -0600 |
|---|---|---|
| committer | Joel Sherrill <joel.sherrill@oarcorp.com> | 2013-12-09 19:50:55 -0600 |
| commit | 967c14385b9194d4257bc44bf21bf1312f9f3321 (patch) | |
| tree | 10aa6562dc3f0e71741798b166fb6267d638cba0 /cpukit/posix | |
| parent | pthread.c: Use UINT32_MAX rather than constant value (diff) | |
| download | rtems-967c14385b9194d4257bc44bf21bf1312f9f3321.tar.bz2 | |
alarm.c: Per comment add debug code for kill() failing
This is highly unlikely and would indicate a serious bug
in the system or corruption. But it is better to be cautious.
Diffstat (limited to 'cpukit/posix')
| -rw-r--r-- | cpukit/posix/src/alarm.c | 21 |
1 files changed, 17 insertions, 4 deletions
diff --git a/cpukit/posix/src/alarm.c b/cpukit/posix/src/alarm.c index d46d571d98..8def4a0836 100644 --- a/cpukit/posix/src/alarm.c +++ b/cpukit/posix/src/alarm.c @@ -9,7 +9,7 @@ * 3.4.1 Schedule Alarm, P1003.1b-1993, p. 79 */ -/* COPYRIGHT (c) 1989-2007. +/* COPYRIGHT (c) 1989-2013. * On-Line Applications Research Corporation (OAR). * * The license and distribution terms for this file may be @@ -32,14 +32,27 @@ /* * _POSIX_signals_Alarm_TSR */ - static void _POSIX_signals_Alarm_TSR( Objects_Id id __attribute__((unused)), void *argument __attribute__((unused)) ) { - kill( getpid(), SIGALRM ); - /* XXX can't print from an ISR, should this be fatal? */ + #if defined(RTEMS_DEBUG) + int status; + #define KILL_STATUS status = + #else + #define KILL_STATUS (void) + #endif + + KILL_STATUS kill( getpid(), SIGALRM ); + + #if defined(RTEMS_DEBUG) + /* + * There is no reason to think this might fail but we should be + * cautious. + */ + assert(status == 0); + #endif } static Watchdog_Control _POSIX_signals_Alarm_timer = WATCHDOG_INITIALIZER( |