From 7fffbdfff7d39cec0783e5b9381fa4093484c235 Mon Sep 17 00:00:00 2001 From: 2012-12-27 Bruno Haible Date: Thu, 27 Dec 2012 22:37:39 +0100 Subject: [PATCH] BZ#14317: Optimze __xpg_strerror_r [BZ #14317] * string/xpg-strerror.c (__xpg_strerror_r): Optimize, call strlen only if needed. --- ChangeLog | 6 ++++++ NEWS | 2 ++ string/xpg-strerror.c | 24 +++++++++++++++--------- 3 files changed, 23 insertions(+), 9 deletions(-) diff --git a/ChangeLog b/ChangeLog index 2a9a361..879fe2d 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,9 @@ +2012-12-27 Bruno Haible + + [BZ #14317] + * string/xpg-strerror.c (__xpg_strerror_r): Optimize, call strlen + only if needed. + 2012-12-27 Siddhesh Poyarekar * sysdeps/ieee754/dbl-64/mpexp.c (__mpexp): Eliminate __mpexp_nn diff --git a/NEWS b/NEWS index c00b555..bae3e41 100644 --- a/NEWS +++ b/NEWS @@ -9,6 +9,8 @@ Version 2.18 * The following bugs are resolved with this release: + 14317. + Version 2.17 diff --git a/string/xpg-strerror.c b/string/xpg-strerror.c index 7e46b33..73600fb 100644 --- a/string/xpg-strerror.c +++ b/string/xpg-strerror.c @@ -1,5 +1,4 @@ -/* Copyright (C) 1991, 1993, 1995-1998, 2000, 2002, 2004, 2010, 2011 - Free Software Foundation, Inc. +/* Copyright (C) 1991-2012 Free Software Foundation, Inc. This file is part of the GNU C Library. The GNU C Library is free software; you can redistribute it and/or @@ -28,20 +27,27 @@ int __xpg_strerror_r (int errnum, char *buf, size_t buflen) { const char *estr = __strerror_r (errnum, buf, buflen); - size_t estrlen = strlen (estr); + /* We know that __strerror_r returns buf (with a dynamically computed + string) if errnum is invalid, otherwise it returns a string whose + storage has indefinite extent. */ if (estr == buf) { assert (errnum < 0 || errnum >= _sys_nerr_internal || _sys_errlist_internal[errnum] == NULL); return EINVAL; } - assert (errnum >= 0 && errnum < _sys_nerr_internal - && _sys_errlist_internal[errnum] != NULL); + else + { + assert (errnum >= 0 && errnum < _sys_nerr_internal + && _sys_errlist_internal[errnum] != NULL); + + size_t estrlen = strlen (estr); - /* Terminate the string in any case. */ - if (buflen > 0) - *((char *) __mempcpy (buf, estr, MIN (buflen - 1, estrlen))) = '\0'; + /* Terminate the string in any case. */ + if (buflen > 0) + *((char *) __mempcpy (buf, estr, MIN (buflen - 1, estrlen))) = '\0'; - return buflen <= estrlen ? ERANGE : 0; + return buflen <= estrlen ? ERANGE : 0; + } } -- 2.7.4