diff options
author | Ludovic Courtès <ludo@gnu.org> | 2008-11-30 18:43:41 +0100 |
---|---|---|
committer | Ludovic Courtès <ludo@gnu.org> | 2008-11-30 18:43:41 +0100 |
commit | 634aa8de8f8facc285867100c6c0a54c4ea5e6d4 (patch) | |
tree | 589bacf6d22ddfe901c5c746220948e0c3e6d04b /libguile/async.c | |
parent | 9b36a80c79c6c26e8e3016eb5d88ea86f66de368 (diff) | |
download | guile-634aa8de8f8facc285867100c6c0a54c4ea5e6d4.tar.gz |
Check the return value of libc's functions to make `_FORTIFY_SOURCE=2' work.
This fixes bug #24009 reported by Martin Pitt.
* libguile/threads.c (guilify_self_1): Check the return value of
pipe(2).
(scm_std_select): Use `full_read ()' instead of `read ()' when reading
from WAKEUP_FD.
* libguile/async.c (scm_i_queue_async_cell): Use `full_write ()' instead
of write(2) when writing to SLEEP_FD.
* libguile/fports.c (fport_flush): Likewise.
* libguile/posix.c (getgroups): Use the return value of getgroups(2) as
NGROUPS.
(scm_nice): Get the return value of nice(2) to make glibc happy.
* libguile/scmsigs.c (take_signal): Use `full_write ()' instead of
write(2).
Diffstat (limited to 'libguile/async.c')
-rw-r--r-- | libguile/async.c | 10 |
1 files changed, 6 insertions, 4 deletions
diff --git a/libguile/async.c b/libguile/async.c index 87d330d60..bf03c48bd 100644 --- a/libguile/async.c +++ b/libguile/async.c @@ -1,4 +1,4 @@ -/* Copyright (C) 1995,1996,1997,1998,2000,2001, 2002, 2004, 2006 Free Software Foundation, Inc. +/* Copyright (C) 1995,1996,1997,1998,2000,2001, 2002, 2004, 2006, 2008 Free Software Foundation, Inc. * * This library is free software; you can redistribute it and/or * modify it under the terms of the GNU Lesser General Public @@ -41,6 +41,8 @@ #include <unistd.h> #endif +#include <full-write.h> + /* {Asynchronous Events} * @@ -241,13 +243,13 @@ scm_i_queue_async_cell (SCM c, scm_i_thread *t) if (sleep_fd >= 0) { char dummy = 0; + /* Likewise, T might already been done with sleeping here, but interrupting it once too often does no harm. T might also not yet have started sleeping, but this is no problem either since the data written to a pipe will not be lost, unlike a - condition variable signal. - */ - write (sleep_fd, &dummy, 1); + condition variable signal. */ + full_write (sleep_fd, &dummy, 1); } /* This is needed to protect sleep_mutex. |