X-Git-Url: http://erislabs.net/gitweb/?a=blobdiff_plain;f=lib%2Fgetdelim.c;h=35cb2d873749a665fc5c432e39ca40884d173c55;hb=545a6f5aa8a1fd105e6f82b7ee9e80a23c072e33;hp=02bb9a49769354b8250af1cdc5109064d641cafd;hpb=f5e93ef0613280d12d81460251dc3084d12beb28;p=gnulib.git diff --git a/lib/getdelim.c b/lib/getdelim.c index 02bb9a497..35cb2d873 100644 --- a/lib/getdelim.c +++ b/lib/getdelim.c @@ -1,5 +1,5 @@ /* getdelim.c --- Implementation of replacement getdelim function. - Copyright (C) 1994, 1996, 1997, 1998, 2001, 2003, 2005 Free + Copyright (C) 1994, 1996, 1997, 1998, 2001, 2003, 2005, 2006, 2007, 2008 Free Software Foundation, Inc. This program is free software; you can redistribute it and/or @@ -19,22 +19,32 @@ /* Ported from glibc by Simon Josefsson. */ -#if HAVE_CONFIG_H -# include -#endif +#include + +#include +#include #include #include -#include "getdelim.h" +#ifndef SIZE_MAX +# define SIZE_MAX ((size_t) -1) +#endif +#ifndef SSIZE_MAX +# define SSIZE_MAX ((ssize_t) (SIZE_MAX / 2)) +#endif -#if !HAVE_FLOCKFILE +#if USE_UNLOCKED_IO +# include "unlocked-io.h" +# define getc_maybe_unlocked(fp) getc(fp) +#elif !HAVE_FLOCKFILE || !HAVE_FUNLOCKFILE || !HAVE_DECL_GETC_UNLOCKED # undef flockfile -# define flockfile(x) ((void) 0) -#endif -#if !HAVE_FUNLOCKFILE # undef funlockfile +# define flockfile(x) ((void) 0) # define funlockfile(x) ((void) 0) +# define getc_maybe_unlocked(fp) getc(fp) +#else +# define getc_maybe_unlocked(fp) getc_unlocked(fp) #endif /* Read up to (and including) a DELIMITER from FP into *LINEPTR (and @@ -46,9 +56,8 @@ ssize_t getdelim (char **lineptr, size_t *n, int delimiter, FILE *fp) { - int result; - ssize_t cur_len = 0; - ssize_t len; + ssize_t result; + size_t cur_len = 0; if (lineptr == NULL || n == NULL || fp == NULL) { @@ -60,33 +69,42 @@ getdelim (char **lineptr, size_t *n, int delimiter, FILE *fp) if (*lineptr == NULL || *n == 0) { + char *new_lineptr; *n = 120; - *lineptr = (char *) malloc (*n); - if (*lineptr == NULL) + new_lineptr = (char *) realloc (*lineptr, *n); + if (new_lineptr == NULL) { result = -1; goto unlock_return; } + *lineptr = new_lineptr; } for (;;) { - char *t; int i; - i = getc (fp); + i = getc_maybe_unlocked (fp); if (i == EOF) - break; + { + result = -1; + break; + } /* Make enough space for len+1 (for final NUL) bytes. */ if (cur_len + 1 >= *n) { - size_t needed = 2 * (cur_len + 1) + 1; /* Be generous. */ + size_t needed_max = + SSIZE_MAX < SIZE_MAX ? (size_t) SSIZE_MAX + 1 : SIZE_MAX; + size_t needed = 2 * *n + 1; /* Be generous. */ char *new_lineptr; - if (needed < cur_len) + if (needed_max < needed) + needed = needed_max; + if (cur_len + 1 >= needed) { result = -1; + errno = EOVERFLOW; goto unlock_return; } @@ -108,9 +126,10 @@ getdelim (char **lineptr, size_t *n, int delimiter, FILE *fp) break; } (*lineptr)[cur_len] = '\0'; - result = cur_len; + result = cur_len ? cur_len : result; unlock_return: - funlockfile (fp); + funlockfile (fp); /* doesn't set errno */ + return result; }