mirror of
git://sourceware.org/git/glibc.git
synced 2024-11-21 01:12:26 +08:00
2cfbdb9a27
Use of strftime, a C90 function, ends up bringing in wcschr, which is not a C90 function. Although not a conformance bug (C90 reserves wcs*), this is still contrary to glibc practice of avoiding relying on those reservations; this patch arranges for the internal uses to use __wcschr instead, with wcschr being a weak alias. This is more complicated than some such patches because of the various IFUNC definitions of wcschr (which include code redefining libc_hidden_def in a way that involves creating __GI_wcschr manually and so also needs to create __GI___wcschr after the change of internal uses to use __wcschr). Tested for x86_64 and 32-bit x86 (testsuite, and that disassembly of installed shared libraries is unchanged by the patch). 2014-12-10 Joseph Myers <joseph@codesourcery.com> Adhemerval Zanella <azanella@linux.vnet.ibm.com> [BZ #17634] * wcsmbs/wcschr.c [!WCSCHR] (wcschr): Define as __wcschr. Undefine after defining function. Define as weak alias of __wcschr. Use libc_hidden_weak. * include/wchar.h (__wcschr): Declare. Use libc_hidden_proto. * sysdeps/i386/i686/multiarch/wcschr-c.c [IS_IN (libc) && SHARED] (libc_hidden_def): Also define __GI___wcschr alias. * sysdeps/i386/i686/multiarch/wcschr.S (wcschr): Rename to __wcschr and define as weak alias of __wcschr. * sysdeps/powerpc/power6/wcschr.c [!WCSCHR] (WCSCHR): Define as __wcschr. [!WCSCHR] (DEFAULT_WCSCHR): Define. [DEFAULT_WCSCHR] (__wcschr): Use libc_hidden_def. [DEFAULT_WCSCHR] (wcschr): Define as weak alias of __wcschr. Use libc_hidden_weak. Do not use libc_hidden_def. * sysdeps/powerpc/powerpc32/power4/multiarch/wcschr-ppc32.c [IS_IN (libc) && SHARED] (libc_hidden_def): Also define __GI___wcschr alias. * sysdeps/powerpc/powerpc32/power4/multiarch/wcschr.c [IS_IN (libc)] (wcschr): Define as macro expanding to __redirect_wcschr. [IS_IN (libc)] (__wcschr_ppc): Use __redirect_wcschr in typeof. [IS_IN (libc)] (__wcschr_power6): Likewise. [IS_IN (libc)] (__wcschr_power7): Likewise. [IS_IN (libc)] (__libc_wcschr): New. Define with libc_ifunc instead of wcschr. [IS_IN (libc)] (wcschr): Undefine and define as weak alias of __libc_wcschr. [!IS_IN (libc)] (libc_hidden_def): Do not undefine and redefine. * sysdeps/powerpc/powerpc64/multiarch/wcschr.c (wcschr): Rename to __wcschr and define as weak alias of __wcschr. Use libc_hidden_builtin_def. * sysdeps/x86_64/wcschr.S (wcschr): Rename to __wcschr and define as weak alias of __wcschr. Use libc_hidden_weak. * time/alt_digit.c (_nl_get_walt_digit): Use __wcschr instead of wcschr. * time/era.c (_nl_init_era_entries): Likewise. * conform/Makefile (test-xfail-ISO/time.h/linknamespace): Remove variable. (test-xfail-XPG3/time.h/linknamespace): Likewise. (test-xfail-XPG4/time.h/linknamespace): Likewise.
194 lines
5.1 KiB
C
194 lines
5.1 KiB
C
/* Helper functions used by strftime/strptime to handle alternate digits.
|
|
Copyright (C) 1995-2014 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
|
|
modify it under the terms of the GNU Lesser General Public
|
|
License as published by the Free Software Foundation; either
|
|
version 2.1 of the License, or (at your option) any later version.
|
|
|
|
The GNU C Library is distributed in the hope that it will be useful,
|
|
but WITHOUT ANY WARRANTY; without even the implied warranty of
|
|
MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
|
|
Lesser General Public License for more details.
|
|
|
|
You should have received a copy of the GNU Lesser General Public
|
|
License along with the GNU C Library; if not, see
|
|
<http://www.gnu.org/licenses/>. */
|
|
|
|
#include "../locale/localeinfo.h"
|
|
#include <bits/libc-lock.h>
|
|
#include <stdlib.h>
|
|
#include <wchar.h>
|
|
#include <string.h>
|
|
#include <stdint.h>
|
|
|
|
/* Some of the functions here must not be used while setlocale is called. */
|
|
__libc_rwlock_define (extern, __libc_setlocale_lock attribute_hidden)
|
|
|
|
#define CURRENT(item) (current->values[_NL_ITEM_INDEX (item)].string)
|
|
#define CURRENT_WSTR(item) \
|
|
((wchar_t *) current->values[_NL_ITEM_INDEX (item)].wstr)
|
|
|
|
static void
|
|
_nl_init_alt_digit (struct __locale_data *current)
|
|
{
|
|
struct lc_time_data *data;
|
|
|
|
if (current->private.time == NULL)
|
|
{
|
|
current->private.time = malloc (sizeof *current->private.time);
|
|
if (current->private.time == NULL)
|
|
return;
|
|
memset (current->private.time, 0, sizeof *current->private.time);
|
|
current->private.cleanup = &_nl_cleanup_time;
|
|
}
|
|
data = current->private.time;
|
|
|
|
if (! data->alt_digits_initialized)
|
|
{
|
|
const char *ptr = CURRENT (ALT_DIGITS);
|
|
size_t cnt;
|
|
|
|
data->alt_digits_initialized = 1;
|
|
|
|
if (ptr != NULL)
|
|
{
|
|
data->alt_digits = malloc (100 * sizeof (const char *));
|
|
if (data->alt_digits != NULL)
|
|
for (cnt = 0; cnt < 100; ++cnt)
|
|
{
|
|
data->alt_digits[cnt] = ptr;
|
|
|
|
/* Skip digit format. */
|
|
ptr = strchr (ptr, '\0') + 1;
|
|
}
|
|
}
|
|
}
|
|
|
|
}
|
|
|
|
const char *
|
|
internal_function
|
|
_nl_get_alt_digit (unsigned int number, struct __locale_data *current)
|
|
{
|
|
const char *result;
|
|
|
|
if (number >= 100 || CURRENT (ALT_DIGITS)[0] == '\0')
|
|
return NULL;
|
|
|
|
__libc_rwlock_wrlock (__libc_setlocale_lock);
|
|
|
|
if (current->private.time == NULL
|
|
|| ! current->private.time->alt_digits_initialized)
|
|
_nl_init_alt_digit (current);
|
|
|
|
result = ((current->private.time != NULL
|
|
&& current->private.time->alt_digits != NULL)
|
|
? current->private.time->alt_digits[number]
|
|
: NULL);
|
|
|
|
__libc_rwlock_unlock (__libc_setlocale_lock);
|
|
|
|
return result;
|
|
}
|
|
|
|
|
|
const wchar_t *
|
|
internal_function
|
|
_nl_get_walt_digit (unsigned int number, struct __locale_data *current)
|
|
{
|
|
const wchar_t *result = NULL;
|
|
struct lc_time_data *data;
|
|
|
|
if (number >= 100 || CURRENT_WSTR (_NL_WALT_DIGITS)[0] == L'\0')
|
|
return NULL;
|
|
|
|
__libc_rwlock_wrlock (__libc_setlocale_lock);
|
|
|
|
if (current->private.time == NULL)
|
|
{
|
|
current->private.time = malloc (sizeof *current->private.time);
|
|
if (current->private.time == NULL)
|
|
goto out;
|
|
memset (current->private.time, 0, sizeof *current->private.time);
|
|
current->private.cleanup = &_nl_cleanup_time;
|
|
}
|
|
data = current->private.time;
|
|
|
|
if (! data->walt_digits_initialized)
|
|
{
|
|
const wchar_t *ptr = CURRENT_WSTR (_NL_WALT_DIGITS);
|
|
size_t cnt;
|
|
|
|
data->walt_digits_initialized = 1;
|
|
|
|
if (ptr != NULL)
|
|
{
|
|
data->walt_digits = malloc (100 * sizeof (const uint32_t *));
|
|
if (data->walt_digits != NULL)
|
|
for (cnt = 0; cnt < 100; ++cnt)
|
|
{
|
|
data->walt_digits[cnt] = ptr;
|
|
|
|
/* Skip digit format. */
|
|
ptr = __wcschr (ptr, L'\0') + 1;
|
|
}
|
|
}
|
|
}
|
|
|
|
if (data->walt_digits != NULL)
|
|
result = data->walt_digits[number];
|
|
|
|
out:
|
|
__libc_rwlock_unlock (__libc_setlocale_lock);
|
|
|
|
return (wchar_t *) result;
|
|
}
|
|
|
|
|
|
int
|
|
internal_function
|
|
_nl_parse_alt_digit (const char **strp, struct __locale_data *current)
|
|
{
|
|
const char *str = *strp;
|
|
int result = -1;
|
|
size_t cnt;
|
|
size_t maxlen = 0;
|
|
|
|
if (CURRENT_WSTR (_NL_WALT_DIGITS)[0] == L'\0')
|
|
return result;
|
|
|
|
__libc_rwlock_wrlock (__libc_setlocale_lock);
|
|
|
|
if (current->private.time == NULL
|
|
|| ! current->private.time->alt_digits_initialized)
|
|
_nl_init_alt_digit (current);
|
|
|
|
if (current->private.time != NULL &&
|
|
current->private.time->alt_digits != NULL)
|
|
/* Matching is not unambiguous. The alternative digits could be like
|
|
I, II, III, ... and the first one is a substring of the second
|
|
and third. Therefore we must keep on searching until we found
|
|
the longest possible match. Note that this is not specified in
|
|
the standard. */
|
|
for (cnt = 0; cnt < 100; ++cnt)
|
|
{
|
|
const char *const dig = current->private.time->alt_digits[cnt];
|
|
size_t len = strlen (dig);
|
|
|
|
if (len > maxlen && strncmp (dig, str, len) == 0)
|
|
{
|
|
maxlen = len;
|
|
result = (int) cnt;
|
|
}
|
|
}
|
|
|
|
__libc_rwlock_unlock (__libc_setlocale_lock);
|
|
|
|
if (result != -1)
|
|
*strp += maxlen;
|
|
|
|
return result;
|
|
}
|