X-Git-Url: http://erislabs.net/gitweb/?a=blobdiff_plain;f=lib%2Fsafe-read.c;h=c21d1cf194004aa02ca07673d8d89d1f9add95c7;hb=c1ebd28b04d9ecf3abc3fe980eaaac13456ba51e;hp=2e131e1f2c600e49ae52c039916808b982c70320;hpb=14b8b11fa55732fc6ac4e8aaeb787b9a3d7ec6db;p=gnulib.git diff --git a/lib/safe-read.c b/lib/safe-read.c index 2e131e1f2..c21d1cf19 100644 --- a/lib/safe-read.c +++ b/lib/safe-read.c @@ -1,5 +1,5 @@ -/* safe-read.c -- an interface to read that retries after interrupts - Copyright (C) 1993, 1994 Free Software Foundation, Inc. +/* An interface to read and write that retries after interrupts. + Copyright (C) 1993, 1994, 1998, 2002-2003 Free Software Foundation, Inc. This program is free software; you can redistribute it and/or modify it under the terms of the GNU General Public License as published by @@ -12,56 +12,71 @@ GNU General Public License for more details. You should have received a copy of the GNU General Public License - along with this program; if not, write to the Free Software - Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA. - */ + along with this program; if not, write to the Free Software Foundation, + Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA. */ -#ifdef HAVE_CONFIG_H -#if defined (CONFIG_BROKETS) -/* We use instead of "config.h" so that a compilation - using -I. -I$srcdir will use ./config.h rather than $srcdir/config.h - (which it would do because it found this file in $srcdir). */ -#include -#else -#include "config.h" +#if HAVE_CONFIG_H +# include #endif + +/* Specification. */ +#ifdef SAFE_WRITE +# include "safe-write.h" +#else +# include "safe-read.h" #endif +/* Get ssize_t. */ #include - -#ifdef HAVE_UNISTD_H -#include +#if HAVE_UNISTD_H +# include #endif #include -#ifndef STDC_HEADERS +#ifndef errno extern int errno; #endif -/* Read LEN bytes at PTR from descriptor DESC, retrying if interrupted. - Return the actual number of bytes read, zero for EOF, or negative - for an error. */ +#ifdef EINTR +# define IS_EINTR(x) ((x) == EINTR) +#else +# define IS_EINTR(x) 0 +#endif + +#include + +#ifdef SAFE_WRITE +# define safe_rw safe_write +# define rw write +#else +# define safe_rw safe_read +# define rw read +# undef const +# define const /* empty */ +#endif -int -safe_read (desc, ptr, len) - int desc; - char *ptr; - int len; +/* Read(write) up to COUNT bytes at BUF from(to) descriptor FD, retrying if + interrupted. Return the actual number of bytes read(written), zero for EOF, + or SAFE_READ_ERROR(SAFE_WRITE_ERROR) upon error. */ +size_t +safe_rw (int fd, void const *buf, size_t count) { - int n_chars; + ssize_t result; - if (len <= 0) - return len; + /* POSIX limits COUNT to SSIZE_MAX, but we limit it further, requiring + that COUNT <= INT_MAX, to avoid triggering a bug in Tru64 5.1. + When decreasing COUNT, keep the file pointer block-aligned. + Note that in any case, read(write) may succeed, yet read(write) + fewer than COUNT bytes, so the caller must be prepared to handle + partial results. */ + if (count > INT_MAX) + count = INT_MAX & ~8191; -#ifdef EINTR do { - n_chars = read (desc, ptr, len); + result = rw (fd, buf, count); } - while (n_chars < 0 && errno == EINTR); -#else - n_chars = read (desc, ptr, len); -#endif + while (result < 0 && IS_EINTR (errno)); - return n_chars; + return (size_t) result; }