Page MenuHomeFreeBSD

locale: persistent mbstates should be reset when rune locale changes
ClosedPublic

Authored by yuripv on Nov 1 2018, 9:30 AM.
Tags
None
Referenced Files
F103429035: D17796.diff
Sun, Nov 24, 10:07 PM
Unknown Object (File)
Sun, Nov 24, 10:17 AM
Unknown Object (File)
Sat, Nov 23, 11:42 AM
Unknown Object (File)
Fri, Nov 22, 5:08 PM
Unknown Object (File)
Tue, Nov 19, 8:33 AM
Unknown Object (File)
Fri, Nov 15, 3:43 AM
Unknown Object (File)
Wed, Nov 6, 4:59 PM
Unknown Object (File)
Oct 16 2024, 7:37 PM
Subscribers

Details

Summary

The test marked as expected to fail in /usr/tests/lib/libc/locale/mbrtowctest (https://github.com/freebsd/freebsd/blob/7bddbea75ccbb4c0d4a2ca5b0e8ce713a878b016/contrib/netbsd-tests/lib/libc/locale/t_mbrtowc.c#L242) seems to be showing a real issue -- when changing ctype component of locale, we can have a state that is perfectly normal for previous locale encoding, and is invalid for new one, leading to unexpected conversion failure. More so, when the locale encoding changes, the meaning of data in opaque structure __mbstate changes as well (every encoding seems to have its own structure). All this lead me to this change resetting the persistent mbstates when new rune locale data is loaded, which required moving the fields from struct _xlocale to its ctype component.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable