musl - musl - an implementation of the standard library for Linux-based systems

musl - an implementation of the standard library for Linux-based systems

diff options

author	Jens Gustedt <Jens.Gustedt@inria.fr>	2014年11月09日 11:18:08 +0100
committer	Rich Felker <dalias@aerifal.cx>	2014年11月15日 15:08:03 -0500
commit	941644e98c3d05761b4639a8ae5afacd8586d1b9 (patch)
tree	07b5a816a539376dde70db9ffcfb739bb59bfcf3 /src
parent	b91cdbe2bc8b626aa04dc6e3e84345accf34e4b1 (diff)
download	musl-941644e98c3d05761b4639a8ae5afacd8586d1b9.tar.gz

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.

Diffstat (limited to 'src')

-rw-r--r--

src/multibyte/c16rtomb.c

-rw-r--r--

src/multibyte/mbrtoc16.c

-rw-r--r--

src/multibyte/mbrtoc32.c

3 files changed, 6 insertions, 0 deletions

diff --git a/src/multibyte/c16rtomb.c b/src/multibyte/c16rtomb.c
index 2e8ec970..39ca3758 100644
--- a/src/multibyte/c16rtomb.c
+++ b/src/multibyte/c16rtomb.c

@@ -4,6 +4,8 @@

size_t c16rtomb(char *restrict s, char16_t c16, mbstate_t *restrict ps)

{

+ static unsigned internal_state;

+ if (!ps) ps = (void *)&internal_state;

unsigned *x = (unsigned *)ps;

wchar_t wc;

diff --git a/src/multibyte/mbrtoc16.c b/src/multibyte/mbrtoc16.c
index 74b7d77e..765ff903 100644
--- a/src/multibyte/mbrtoc16.c
+++ b/src/multibyte/mbrtoc16.c

@@ -3,6 +3,8 @@

size_t mbrtoc16(char16_t *restrict pc16, const char *restrict s, size_t n, mbstate_t *restrict ps)

{

+ static unsigned internal_state;

+ if (!ps) ps = (void *)&internal_state;

unsigned *pending = (unsigned *)ps;

if (!s) return mbrtoc16(0, "", 1, ps);

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);

generated by cgit v1.2.1 (git 2.18.0) at 2025年10月06日 01:20:18 +0000