From 941644e98c3d05761b4639a8ae5afacd8586d1b9 Mon Sep 17 00:00:00 2001 From: Jens Gustedt Date: Sun, 9 Nov 2014 11:18:08 +0100 Subject: implement a private state for the uchar.h functions The C standard is imperative on that: 7.28.1 ... If ps is a null pointer, each function uses its own internal mbstate_t object instead, which is initialized at program startup to the initial conversion state; and these functions are also not supposed to implicitly use the state of the wchar.h functions: 7.29.6.3 ... The implementation behaves as if no library function calls these functions with a null pointer for ps. Previously this resulted in two bugs. - The functions c16rtomb and mbrtoc16 would crash when called with ps set to null. - The function mbrtoc32 used the private state of mbrtowc, which it is not allowed to do. --- src/multibyte/mbrtoc32.c | 2 ++ 1 file changed, 2 insertions(+) (limited to 'src/multibyte/mbrtoc32.c') diff --git a/src/multibyte/mbrtoc32.c b/src/multibyte/mbrtoc32.c index c6d20824..9b6b2367 100644 --- a/src/multibyte/mbrtoc32.c +++ b/src/multibyte/mbrtoc32.c @@ -3,6 +3,8 @@ size_t mbrtoc32(char32_t *restrict pc32, const char *restrict s, size_t n, mbstate_t *restrict ps) { + static unsigned internal_state; + if (!ps) ps = (void *)&internal_state; if (!s) return mbrtoc32(0, "", 1, ps); wchar_t wc; size_t ret = mbrtowc(&wc, s, n, ps); -- cgit v1.2.1