X-Git-Url: http://erislabs.net/gitweb/?a=blobdiff_plain;f=lib%2Fsnprintf.c;h=db1ca9af1e7c2ab4e9f0a55073f5345dfe76084a;hb=74fcb0575a5b053a1b1078a5595c1795790c63cb;hp=5870f8dde30f08250f6d78a1683cc14c034aaa27;hpb=267a39bafd249d7eb9c37df06dc6defcf41cb343;p=gnulib.git diff --git a/lib/snprintf.c b/lib/snprintf.c index 5870f8dde..db1ca9af1 100644 --- a/lib/snprintf.c +++ b/lib/snprintf.c @@ -1,6 +1,6 @@ /* Formatted output to strings. - Copyright (C) 2004 Free Software Foundation, Inc. - Written by Simon Josefsson. + Copyright (C) 2004, 2006 Free Software Foundation, Inc. + Written by Simon Josefsson and Paul Eggert. 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 @@ -16,19 +16,23 @@ with this program; if not, write to the Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA. */ -#ifdef HAVE_CONFIG_H -# include -#endif +#include #include "snprintf.h" +#include +#include #include #include #include -#include "minmax.h" #include "vasnprintf.h" +/* Some systems, like OSF/1 4.0 and Woe32, don't have EOVERFLOW. */ +#ifndef EOVERFLOW +# define EOVERFLOW E2BIG +#endif + /* Print formatted output to string STR. Similar to sprintf, but additional length SIZE limit how much is written into STR. Returns string length of formatted string (which may be larger than SIZE). @@ -39,22 +43,34 @@ snprintf (char *str, size_t size, const char *format, ...) { char *output; size_t len; + size_t lenbuf = size; va_list args; va_start (args, format); - len = size; - output = vasnprintf (str, &len, format, args); + output = vasnprintf (str, &lenbuf, format, args); + len = lenbuf; va_end (args); if (!output) return -1; - if (str != NULL) - if (len > size - 1) /* equivalent to: (size > 0 && len >= size) */ - str[size - 1] = '\0'; - if (output != str) - free (output); + { + if (size) + { + size_t pruned_len = (len < size ? len : size - 1); + memcpy (str, output, pruned_len); + str[pruned_len] = '\0'; + } + + free (output); + } + + if (INT_MAX < len) + { + errno = EOVERFLOW; + return -1; + } return len; }