From 727211c42911512f71d23eacf584ce1c3dbfd4c2 Mon Sep 17 00:00:00 2001 From: Ulrich Drepper Date: Mon, 8 Nov 1999 23:44:12 +0000 Subject: [PATCH] Update. 1999-11-08 Andreas Jaeger * locale/findlocale.c (_nl_find_locale): Free normalized_codeset; this fixes a memory leak (closes PR libc/1435). * intl/l10nflist.c (_nl_normalize_codeset): Update comment to reflect memory allocation. * intl/loadinfo.h: Comment _nl_normalize_codeset, comment memory allocation in _nl_explode_name. 1999-11-06 Andreas Jaeger * time/tzset.c: Use __libc_lock_define_initialized for tzset_lock to initialize it. Reported by wes@surety.com, closes PR libc/1432. 1999-11-05 Geoff Keating * sysdeps/powerpc/fpu/bits/fenv.h: Don't put commas at the end of enumerator lists (-pedantic complains). (feraiseexcept): Use i#*X as the constraint. Fix a bug in the test for 'one bit set'. (feclearexcept): Likewise. --- ChangeLog | 25 ++++++++++++++++++++ intl/l10nflist.c | 3 ++- intl/loadinfo.h | 6 +++++ locale/findlocale.c | 4 ++++ sysdeps/powerpc/fpu/bits/fenv.h | 46 +++++++++++++++++++++---------------- sysdeps/unix/sysv/linux/setrlimit.c | 2 +- time/tzset.c | 2 +- 7 files changed, 65 insertions(+), 23 deletions(-) diff --git a/ChangeLog b/ChangeLog index f06e03a..f9bfe76 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,28 @@ +1999-11-08 Andreas Jaeger + + * locale/findlocale.c (_nl_find_locale): Free normalized_codeset; + this fixes a memory leak (closes PR libc/1435). + + * intl/l10nflist.c (_nl_normalize_codeset): Update comment to + reflect memory allocation. + + * intl/loadinfo.h: Comment _nl_normalize_codeset, comment memory + allocation in _nl_explode_name. + +1999-11-06 Andreas Jaeger + + * time/tzset.c: Use __libc_lock_define_initialized for tzset_lock + to initialize it. + Reported by wes@surety.com, closes PR libc/1432. + +1999-11-05 Geoff Keating + + * sysdeps/powerpc/fpu/bits/fenv.h: Don't put commas at the end of + enumerator lists (-pedantic complains). + (feraiseexcept): Use i#*X as the constraint. Fix a bug in the + test for 'one bit set'. + (feclearexcept): Likewise. + 1999-11-05 Ulrich Drepper * sysdeps/unix/sysv/linux/setrlimit.c: New file. diff --git a/intl/l10nflist.c b/intl/l10nflist.c index 5edba3a..871b428 100644 --- a/intl/l10nflist.c +++ b/intl/l10nflist.c @@ -352,7 +352,8 @@ _nl_make_l10nflist (l10nfile_list, dirlist, dirlist_len, mask, language, /* Normalize codeset name. There is no standard for the codeset names. Normalization allows the user to use any of the common - names. */ + names. The return value is dynamically allocated and has to be + freed by the caller. */ const char * _nl_normalize_codeset (codeset, name_len) const char *codeset; diff --git a/intl/loadinfo.h b/intl/loadinfo.h index ea1bf05..09b2fdf 100644 --- a/intl/loadinfo.h +++ b/intl/loadinfo.h @@ -54,6 +54,10 @@ struct loaded_l10nfile }; +/* Normalize codeset name. There is no standard for the codeset + names. Normalization allows the user to use any of the common + names. The return value is dynamically allocated and has to be + freed by the caller. */ extern const char *_nl_normalize_codeset PARAMS ((const char *codeset, size_t name_len)); @@ -70,6 +74,8 @@ _nl_make_l10nflist PARAMS ((struct loaded_l10nfile **l10nfile_list, extern const char *_nl_expand_alias PARAMS ((const char *name)); +/* normalized_codeset is dynamically allocated and has to be freed by + the caller. */ extern int _nl_explode_name PARAMS ((char *name, const char **language, const char **modifier, const char **territory, diff --git a/locale/findlocale.c b/locale/findlocale.c index c6717f1..9cb2206 100644 --- a/locale/findlocale.c +++ b/locale/findlocale.c @@ -141,6 +141,10 @@ _nl_find_locale (const char *locale_path, size_t locale_path_len, between two locales would slowly eat up all memory. */ free ((void *) loc_name); + /* The space for normalized_codeset is dynamically allocated. Free it. */ + if (mask & XPG_NORM_CODESET) + free ((void *) normalized_codeset); + if (locale_file->decided == 0) _nl_load_locale (locale_file, category); diff --git a/sysdeps/powerpc/fpu/bits/fenv.h b/sysdeps/powerpc/fpu/bits/fenv.h index f86d02d..459821f 100644 --- a/sysdeps/powerpc/fpu/bits/fenv.h +++ b/sysdeps/powerpc/fpu/bits/fenv.h @@ -86,7 +86,7 @@ enum # define FE_INVALID_SQRT FE_INVALID_SQRT /* Conversion-to-integer of a NaN or a number too large or too small. */ - FE_INVALID_INTEGER_CONVERSION = 1 << (31 - 23), + FE_INVALID_INTEGER_CONVERSION = 1 << (31 - 23) # define FE_INVALID_INTEGER_CONVERSION FE_INVALID_INTEGER_CONVERSION # define FE_ALL_INVALID \ @@ -110,7 +110,7 @@ enum #define FE_TOWARDZERO FE_TOWARDZERO FE_UPWARD = 2, #define FE_UPWARD FE_UPWARD - FE_DOWNWARD = 3, + FE_DOWNWARD = 3 #define FE_DOWNWARD FE_DOWNWARD }; @@ -152,27 +152,33 @@ extern const fenv_t *__fe_nomask_env __P ((void)); : "=r"(__fegetround_result) : : "cr7"); \ __fegetround_result & 3; })) +/* The weird 'i#*X' constraints on the following suppress a gcc + warning when __excepts is not a constant. Otherwise, they mean the + same as just plain 'i'. */ + /* Inline definition for feraiseexcept. */ # define feraiseexcept(__excepts) \ - (__extension__ ({ if (__builtin_constant_p (__excepts) \ - && ((__excepts) & -(__excepts)) == 0 \ - && (__excepts) != FE_INVALID) { \ - if ((__excepts) != 0) \ - __asm__ __volatile__ \ - ("mtfsb1 %0" \ - : : "i"(32 - __builtin_ffs (__excepts))); \ - } else \ - (feraiseexcept) (__excepts); })) + ((__builtin_constant_p (__excepts) \ + && ((__excepts) & ((__excepts)-1)) == 0 \ + && (__excepts) != FE_INVALID) \ + ? ((__excepts) != 0 \ + ? (__extension__ ({ __asm__ __volatile__ \ + ("mtfsb1 %s0" \ + : : "i#*X"(__builtin_ffs (__excepts))); \ + (void)0; })) \ + : (void)0) \ + : (feraiseexcept) (__excepts)) /* Inline definition for feclearexcept. */ # define feclearexcept(__excepts) \ - (__extension__ ({ if (__builtin_constant_p (__excepts) \ - && ((__excepts) & -(__excepts)) == 0 \ - && (__excepts) != FE_INVALID) { \ - if ((__excepts) != 0) \ - __asm__ __volatile__ \ - ("mtfsb0 %0" \ - : : "i"(32 - __builtin_ffs (__excepts))); \ - } else \ - (feclearexcept) (__excepts); })) + ((__builtin_constant_p (__excepts) \ + && ((__excepts) & ((__excepts)-1)) == 0 \ + && (__excepts) != FE_INVALID) \ + ? ((__excepts) != 0 \ + ? (__extension__ ({ __asm__ __volatile__ \ + ("mtfsb0 %s0" \ + : : "i#*X"(__builtin_ffs (__excepts))); \ + (void)0; })) \ + : (void)0) \ + : (feclearexcept) (__excepts)) #endif /* __OPTIMIZE__ && !_SOFT_FLOAT */ diff --git a/sysdeps/unix/sysv/linux/setrlimit.c b/sysdeps/unix/sysv/linux/setrlimit.c index 674a2cf..1316159 100644 --- a/sysdeps/unix/sysv/linux/setrlimit.c +++ b/sysdeps/unix/sysv/linux/setrlimit.c @@ -23,7 +23,7 @@ #include #include -#include "../kernel-features.h" +#include "kernel-features.h" /* Linux 2.3.25 introduced a new system call since the types used for diff --git a/time/tzset.c b/time/tzset.c index 6af2fba..db08a47 100644 --- a/time/tzset.c +++ b/time/tzset.c @@ -38,7 +38,7 @@ weak_alias (__daylight, daylight) weak_alias (__timezone, timezone) /* This locks all the state variables in tzfile.c and this file. */ -__libc_lock_define (static, tzset_lock) +__libc_lock_define_initialized (static, tzset_lock) #define min(a, b) ((a) < (b) ? (a) : (b)) -- 2.7.4