fix wide printf forms ignoring width for %lc format specifier

[musl] / src / locale / setlocale.c

diff --git a/src/locale/setlocale.c b/src/locale/setlocale.c
index 11d823c..360c443 100644 (file)
--- a/src/locale/setlocale.c
+++ b/src/locale/setlocale.c
@@ -7,23 +7,13 @@
 
 static char buf[LC_ALL*(LOCALE_NAME_MAX+1)];
 
-static char *setlocale_one_unlocked(int cat, const char *name)
-{
-       const struct __locale_map *lm;
-
-       if (name) libc.global_locale.cat[cat] = lm = __get_locale(cat, name);
-       else lm = libc.global_locale.cat[cat];
-
-       return lm ? (char *)lm->name : "C";
-}
-
 char *setlocale(int cat, const char *name)
 {
-       static volatile int lock[1];
+       const struct __locale_map *lm;
 
        if ((unsigned)cat > LC_ALL) return 0;
 
-       LOCK(lock);
+       LOCK(__locale_lock);
 
        /* For LC_ALL, setlocale is required to return a string which
         * encodes the current setting for all categories. The format of
@@ -33,6 +23,7 @@ char *setlocale(int cat, const char *name)
        if (cat == LC_ALL) {
                int i;
                if (name) {
+                       struct __locale_struct tmp_locale;
                        char part[LOCALE_NAME_MAX+1] = "C.UTF-8";
                        const char *p = name;
                        for (i=0; i<LC_ALL; i++) {
@@ -42,8 +33,14 @@ char *setlocale(int cat, const char *name)
                                        part[z-p] = 0;
                                        if (*z) p = z+1;
                                }
-                               setlocale_one_unlocked(i, part);
+                               lm = __get_locale(i, part);
+                               if (lm == LOC_MAP_FAILED) {
+                                       UNLOCK(__locale_lock);
+                                       return 0;
+                               }
+                               tmp_locale.cat[i] = lm;
                        }
+                       libc.global_locale = tmp_locale;
                }
                char *s = buf;
                const char *part;
@@ -59,13 +56,23 @@ char *setlocale(int cat, const char *name)
                        s += l+1;
                }
                *--s = 0;
-               UNLOCK(lock);
+               UNLOCK(__locale_lock);
                return same==LC_ALL ? (char *)part : buf;
        }
 
-       char *ret = setlocale_one_unlocked(cat, name);
+       if (name) {
+               lm = __get_locale(cat, name);
+               if (lm == LOC_MAP_FAILED) {
+                       UNLOCK(__locale_lock);
+                       return 0;
+               }
+               libc.global_locale.cat[cat] = lm;
+       } else {
+               lm = libc.global_locale.cat[cat];
+       }
+       char *ret = lm ? (char *)lm->name : "C";
 
-       UNLOCK(lock);
+       UNLOCK(__locale_lock);
 
        return ret;
 }