diff options
| author | Ulrich Drepper <drepper@redhat.com> | 2009-02-07 04:40:57 +0000 |
|---|---|---|
| committer | Ulrich Drepper <drepper@redhat.com> | 2009-02-07 04:40:57 +0000 |
| commit | 1a9dd4e4e42315b3c744b346e4696d1362b1fc9a (patch) | |
| tree | 74a32492c582e7e6d3b96340f7526a2cafec9849 | |
| parent | 41f128dc0470fd006dbeebd0cb06534bc56a5e16 (diff) | |
| download | glibc-1a9dd4e4e42315b3c744b346e4696d1362b1fc9a.zip glibc-1a9dd4e4e42315b3c744b346e4696d1362b1fc9a.tar.gz glibc-1a9dd4e4e42315b3c744b346e4696d1362b1fc9a.tar.bz2 | |
[BZ #9781]
* grp/compat-initgroups.c (compat_call): Switch to use malloc when
the input line is too long.
| -rw-r--r-- | ChangeLog | 4 | ||||
| -rw-r--r-- | grp/compat-initgroups.c | 26 | ||||
| -rw-r--r-- | localedata/ChangeLog | 7 | ||||
| -rw-r--r-- | localedata/locales/el_CY | 4 | ||||
| -rw-r--r-- | localedata/locales/el_GR | 4 | ||||
| -rw-r--r-- | resolv/res_libc.c | 5 |
6 files changed, 38 insertions, 12 deletions
@@ -1,5 +1,9 @@ 2009-02-06 Ulrich Drepper <drepper@redhat.com> + [BZ #9781] + * grp/compat-initgroups.c (compat_call): Switch to use malloc when + the input line is too long. + * po/Makefile (libc.pot): Add f_print as function taking c-format parameter. diff --git a/grp/compat-initgroups.c b/grp/compat-initgroups.c index efd875a..374f42a 100644 --- a/grp/compat-initgroups.c +++ b/grp/compat-initgroups.c @@ -14,7 +14,6 @@ compat_call (service_user *nip, const char *user, gid_t group, long int *start, { struct group grpbuf; size_t buflen = __sysconf (_SC_GETGR_R_SIZE_MAX); - char *tmpbuf; enum nss_status status; set_function setgrent_fct; get_function getgrent_fct; @@ -35,7 +34,9 @@ compat_call (service_user *nip, const char *user, gid_t group, long int *start, endgrent_fct = __nss_lookup_function (nip, "endgrent"); - tmpbuf = __alloca (buflen); + char *tmpbuf = __alloca (buflen); + bool use_malloc = false; + enum nss_status result = NSS_STATUS_SUCCESS; do { @@ -44,8 +45,20 @@ compat_call (service_user *nip, const char *user, gid_t group, long int *start, status == NSS_STATUS_TRYAGAIN) && *errnop == ERANGE) { - buflen *= 2; - tmpbuf = __alloca (buflen); + if (__libc_use_alloca (buflen * 2)) + tmpbuf = extend_alloca (tmpbuf, buflen, buflen * 2); + else + { + buflen *= 2; + char *newbuf = realloc (use_malloc ? tmpbuf : NULL, buflen); + if (newbuf == NULL) + { + result = NSS_STATUS_TRYAGAIN; + goto done; + } + use_malloc = true; + tmpbuf = newbuf; + } } if (status != NSS_STATUS_SUCCESS) @@ -102,8 +115,11 @@ compat_call (service_user *nip, const char *user, gid_t group, long int *start, while (status == NSS_STATUS_SUCCESS); done: + if (use_malloc) + free (tmpbuf); + if (endgrent_fct) DL_CALL_FCT (endgrent_fct, ()); - return NSS_STATUS_SUCCESS; + return result; } diff --git a/localedata/ChangeLog b/localedata/ChangeLog index c748d0d..12e7e68 100644 --- a/localedata/ChangeLog +++ b/localedata/ChangeLog @@ -1,3 +1,10 @@ +2009-02-06 Ulrich Drepper <drepper@redhat.com> + + [BZ #9736] + * locales/el_CY: Fix frac_digits and int_frac_digits. + * locales/el_GR: Likewise. + Patch by Clint Adams <schizo@debian.org>. + 2009-01-28 Ulrich Drepper <drepper@redhat.com> [BZ #9791] diff --git a/localedata/locales/el_CY b/localedata/locales/el_CY index 7213924..3117577 100644 --- a/localedata/locales/el_CY +++ b/localedata/locales/el_CY @@ -51,8 +51,8 @@ mon_thousands_sep "<U002E>" mon_grouping 3 positive_sign "" negative_sign "<U002D>" -int_frac_digits 4 -frac_digits 4 +int_frac_digits 2 +frac_digits 2 p_cs_precedes 0 p_sep_by_space 0 n_cs_precedes 1 diff --git a/localedata/locales/el_GR b/localedata/locales/el_GR index ce608a8..bb1db14 100644 --- a/localedata/locales/el_GR +++ b/localedata/locales/el_GR @@ -70,8 +70,8 @@ mon_thousands_sep "<U002E>" mon_grouping 3 positive_sign "" negative_sign "<U002D>" -int_frac_digits 4 -frac_digits 4 +int_frac_digits 2 +frac_digits 2 p_cs_precedes 0 p_sep_by_space 0 n_cs_precedes 1 diff --git a/resolv/res_libc.c b/resolv/res_libc.c index 8af57f7..810fbc8 100644 --- a/resolv/res_libc.c +++ b/resolv/res_libc.c @@ -96,10 +96,9 @@ __res_maybe_init (res_state resp, int preinit) { if (resp->options & RES_INIT) { if (__res_initstamp != resp->_u._ext.initstamp) { - if (resp->nscount > 0) { + if (resp->nscount > 0) __res_iclose (resp, true); - return __res_vinit (resp, 1); - } + return __res_vinit (resp, 1); } return 0; } else if (preinit) { |