diff options
| author |   schwarze <schwarze@openbsd.org> | 2018-03-29 16:34:25 +0000 |
|---|---|---|
| committer | schwarze <schwarze@openbsd.org> | 2018-03-29 16:34:25 +0000 |
| commit | f85ded84d67ce8e7f7589305fc9831335a4d3ad4 (patch) | |
| tree | b2dfd99372a7529594cdef7f46d2f2d2c787274e /lib/libc | |
| parent | Add support for legacy binding used in device trees for Marvell devices for (diff) | |
| download | wireguard-openbsd-f85ded84d67ce8e7f7589305fc9831335a4d3ad4.tar.xz wireguard-openbsd-f85ded84d67ce8e7f7589305fc9831335a4d3ad4.zip | |
Fix three bugs in setlocale(3):
1. setlocale(LC_ALL, "A"); setlocale(LC_CTYPE, "T"); setlocale(LC_ALL, NULL);
must return "A/T/A/A/A/A", not "A". Fix this by always initializing the
LC_ALL entry of newgl to "" in dupgl(). Reported by Karl Williamson
<public at khwilliamson dot com> on bugs@, thanks!
2. Do not leak newgl when strdup(3) fails in setlocale(3).
3. For setlocale(LC_ALL, "C/C/fr_FR.UTF-8/C/C/C"); correctly set
_GlobalRuneLocale; i found 2. and 3. while looking at the code.
Feedback on a buggy earlier version and OK martijn@.
Diffstat (limited to 'lib/libc')
| -rw-r--r-- | lib/libc/locale/setlocale.c | 12 |
1 files changed, 7 insertions, 5 deletions
diff --git a/lib/libc/locale/setlocale.c b/lib/libc/locale/setlocale.c index 7afc46ed497..78e5f0a5388 100644 --- a/lib/libc/locale/setlocale.c +++ b/lib/libc/locale/setlocale.c @@ -1,4 +1,4 @@ -/* $OpenBSD: setlocale.c,v 1.27 2017/09/05 03:16:13 schwarze Exp $ */ +/* $OpenBSD: setlocale.c,v 1.28 2018/03/29 16:34:25 schwarze Exp $ */ /* * Copyright (c) 2017 Ingo Schwarze <schwarze@openbsd.org> * @@ -42,8 +42,8 @@ dupgl(char **oldgl) if ((newgl = calloc(_LC_LAST, sizeof(*newgl))) == NULL) return NULL; for (ic = LC_ALL; ic < _LC_LAST; ic++) { - if ((newgl[ic] = strdup(oldgl != NULL ? - oldgl[ic] : ic == LC_ALL ? "" : "C")) == NULL) { + if ((newgl[ic] = strdup(ic == LC_ALL ? "" : + oldgl == NULL ? "C" : oldgl[ic])) == NULL) { freegl(newgl); return NULL; } @@ -92,8 +92,10 @@ setlocale(int category, const char *locname) if (category == LC_ALL && strchr(locname, '/') != NULL) { /* One value for each category. */ - if ((firstname = strdup(locname)) == NULL) + if ((firstname = strdup(locname)) == NULL) { + freegl(newgl); return NULL; + } nextname = firstname; for (ic = 1; ic < _LC_LAST; ic++) if (nextname == NULL || changegl(ic, @@ -184,7 +186,7 @@ done: global_locale = newgl; if (category == LC_ALL || category == LC_CTYPE) _GlobalRuneLocale = - strchr(global_locname, '.') == NULL ? + strchr(newgl[LC_CTYPE], '.') == NULL ? &_DefaultRuneLocale : _Utf8RuneLocale; } else { freegl(newgl); |