X-Git-Url: http://erislabs.net/gitweb/?a=blobdiff_plain;f=lib%2Fxreadlink.c;h=41f5242ea468c947cdb30fcaae3278899823ee9d;hb=10cf9b5e371bc4927b2b28354c19b62943dfe95e;hp=f5cce16eb8f9adf763c0deec7c44556b66feedc2;hpb=dde911213fff46066f0369775a0f554e79b2a8f5;p=gnulib.git diff --git a/lib/xreadlink.c b/lib/xreadlink.c index f5cce16eb..41f5242ea 100644 --- a/lib/xreadlink.c +++ b/lib/xreadlink.c @@ -1,6 +1,6 @@ /* xreadlink.c -- readlink wrapper to return the link name in malloc'd storage - Copyright (C) 2001, 2003 Free Software Foundation, Inc. + Copyright (C) 2001, 2003, 2004 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 @@ -23,6 +23,8 @@ # include #endif +#include "xreadlink.h" + #include #include #ifndef errno @@ -31,9 +33,7 @@ extern int errno; #include #include -#if HAVE_STDLIB_H -# include -#endif +#include #if HAVE_UNISTD_H # include #endif @@ -46,20 +46,21 @@ extern int errno; #endif #include "xalloc.h" -#include "xreadlink.h" /* Call readlink to get the symbolic link value of FILENAME. + SIZE is a hint as to how long the link is expected to be; + typically it is taken from st_size. It need not be correct. Return a pointer to that NUL-terminated string in malloc'd storage. If readlink fails, return NULL (caller may use errno to diagnose). - If realloc fails, or if the link value is longer than SIZE_MAX :-), + If malloc fails, or if the link value is longer than SSIZE_MAX :-), give a diagnostic and exit. */ char * -xreadlink (char const *filename) +xreadlink (char const *filename, size_t size) { /* The initial buffer size for the link value. A power of 2 detects arithmetic overflow earlier, but is not required. */ - size_t buf_size = 128; + size_t buf_size = size + 1; while (1) { @@ -82,7 +83,7 @@ xreadlink (char const *filename) free (buffer); buf_size *= 2; - if (SSIZE_MAX < buf_size || (SIZE_MAX / 2 < SSIZE_MAX && buf_size == 0)) + if (! (0 < buf_size && buf_size <= SSIZE_MAX)) xalloc_die (); } }