X-Git-Url: http://erislabs.net/gitweb/?a=blobdiff_plain;f=lib%2Fstrerror.c;h=63899ca6b264e55bb6e8ab6b5614f5293361a341;hb=cea318a913148d825c03a4e45f4553e4cb4e2880;hp=fd5ec15620694c1705697415b0f25d9d9adff67b;hpb=57fdfd3f8ec62b105c53bcdf6f127c35c7fe7391;p=gnulib.git diff --git a/lib/strerror.c b/lib/strerror.c index fd5ec1562..63899ca6b 100644 --- a/lib/strerror.c +++ b/lib/strerror.c @@ -1,7 +1,6 @@ -/* strerror.c --- ANSI C compatible system error routine +/* strerror.c --- POSIX compatible system error routine - Copyright (C) 1986, 1988, 1989, 1991, 2002, 2003, 2006, 2007 Free - Software Foundation, Inc. + Copyright (C) 2007-2011 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 @@ -18,61 +17,54 @@ #include -#if REPLACE_STRERROR +/* Specification. */ +#include -# include -# include +#include +#include +#include +#include -# undef strerror +#include "intprops.h" +#include "strerror-override.h" +#include "verify.h" -char *rpl_strerror (int n) -{ - static char const fmt[] = "Unknown error (%d)"; - static char mesg[sizeof fmt + sizeof n * CHAR_BIT / 3]; - - char *result = strerror (n); - - if (! result) - { - sprintf (mesg, fmt, n); - return mesg; - } - return result; -} - -#elif !HAVE_STRERROR - -#include - -/* Don't include , since it may or may not declare - sys_errlist and its declarations may collide with ours. Just - declare the stuff that we need directly. Standard hosted C89 - implementations define strerror and they don't need this strerror - function, so take some liberties with the standard to cater to - ancient or limited freestanding implementations. */ -int sprintf (char *, char const *, ...); -extern int sys_nerr; -extern char *sys_errlist[]; +/* Use the system functions, not the gnulib overrides in this file. */ +#undef sprintf char * strerror (int n) +#undef strerror { - static char const fmt[] = "Unknown error (%d)"; - static char mesg[sizeof fmt + sizeof n * CHAR_BIT / 3]; - - if (n < 0 || n >= sys_nerr) + static char buf[STACKBUF_LEN]; + size_t len; + + /* Cast away const, due to the historical signature of strerror; + callers should not be modifying the string. */ + const char *msg = strerror_override (n); + if (msg) + return (char *) msg; + + msg = strerror (n); + + /* Our strerror_r implementation might use the system's strerror + buffer, so all other clients of strerror have to see the error + copied into a buffer that we manage. This is not thread-safe, + even if the system strerror is, but portable programs shouldn't + be using strerror if they care about thread-safety. */ + if (!msg || !*msg) { - sprintf (mesg, fmt, n); - return mesg; + static char const fmt[] = "Unknown error %d"; + verify (sizeof buf >= sizeof (fmt) + INT_STRLEN_BOUND (n)); + sprintf (buf, fmt, n); + errno = EINVAL; + return buf; } - else - return sys_errlist[n]; -} - -#else -/* This declaration is solely to ensure that after preprocessing - this file is never empty. */ -typedef int dummy; + /* Fix STACKBUF_LEN if this ever aborts. */ + len = strlen (msg); + if (sizeof buf <= len) + abort (); -#endif + return memcpy (buf, msg, len + 1); +}