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
Unknown Object (File)
Mon, Dec 2, 1:32 AM
Unknown Object (File)
Sat, Nov 30, 5:27 PM
Unknown Object (File)
Wed, Nov 27, 6:53 PM
Unknown Object (File)
Nov 24 2024, 10:07 PM
Unknown Object (File)
Nov 24 2024, 10:17 AM
Unknown Object (File)
Nov 23 2024, 11:42 AM
Unknown Object (File)
Nov 22 2024, 5:08 PM
Unknown Object (File)
Nov 19 2024, 8:33 AM
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