diff options
author | Victor Stinner <vstinner@redhat.com> | 2018-11-23 12:08:26 (GMT) |
---|---|---|
committer | GitHub <noreply@github.com> | 2018-11-23 12:08:26 (GMT) |
commit | 353933e712b6c7f7ba9a9a50bd5bd472db7c35d0 (patch) | |
tree | b91db22bcb76a9459a0495c96f21524368af401c | |
parent | e89607c0fc8d6edbf19c06ba42ff0f00e6c4273f (diff) | |
download | cpython-353933e712b6c7f7ba9a9a50bd5bd472db7c35d0.zip cpython-353933e712b6c7f7ba9a9a50bd5bd472db7c35d0.tar.gz cpython-353933e712b6c7f7ba9a9a50bd5bd472db7c35d0.tar.bz2 |
bpo-34523: Fix C locale coercion on FreeBSD CURRENT (GH-10672)
bpo-34523, bpo-35290: C locale coercion now resets the Python
internal "force ASCII" mode. This change fix the filesystem encoding
on FreeBSD CURRENT, which has a new "C.UTF-8" locale, when
the UTF-8 mode is disabled.
Add _Py_ResetForceASCII(): _Py_SetLocaleFromEnv() now calls it.
-rw-r--r-- | Include/internal/pycore_fileutils.h | 8 | ||||
-rw-r--r-- | Python/fileutils.c | 13 | ||||
-rw-r--r-- | Python/pylifecycle.c | 12 |
3 files changed, 29 insertions, 4 deletions
diff --git a/Include/internal/pycore_fileutils.h b/Include/internal/pycore_fileutils.h index 98eb258..2500665 100644 --- a/Include/internal/pycore_fileutils.h +++ b/Include/internal/pycore_fileutils.h @@ -32,6 +32,14 @@ PyAPI_FUNC(wchar_t*) _Py_DecodeUTF8_surrogateescape( PyAPI_FUNC(int) _Py_GetForceASCII(void); +/* Reset "force ASCII" mode (if it was initialized). + + This function should be called when Python changes the LC_CTYPE locale, + so the "force ASCII" mode can be detected again on the new locale + encoding. */ +PyAPI_FUNC(void) _Py_ResetForceASCII(void); + + PyAPI_FUNC(int) _Py_GetLocaleconvNumeric( struct lconv *lc, PyObject **decimal_point, diff --git a/Python/fileutils.c b/Python/fileutils.c index 033c2ff..366bd00 100644 --- a/Python/fileutils.c +++ b/Python/fileutils.c @@ -231,6 +231,13 @@ _Py_GetForceASCII(void) } +void +_Py_ResetForceASCII(void) +{ + force_ascii = -1; +} + + static int encode_ascii(const wchar_t *text, char **str, size_t *error_pos, const char **reason, @@ -296,6 +303,12 @@ _Py_GetForceASCII(void) { return 0; } + +void +_Py_ResetForceASCII(void) +{ + /* nothing to do */ +} #endif /* !defined(__APPLE__) && !defined(__ANDROID__) && !defined(MS_WINDOWS) */ diff --git a/Python/pylifecycle.c b/Python/pylifecycle.c index af3d5ef..6de32de 100644 --- a/Python/pylifecycle.c +++ b/Python/pylifecycle.c @@ -5,6 +5,7 @@ #include "Python-ast.h" #undef Yield /* undefine macro conflicting with <winbase.h> */ #include "pycore_context.h" +#include "pycore_fileutils.h" #include "pycore_hamt.h" #include "pycore_pathconfig.h" #include "pycore_pylifecycle.h" @@ -394,6 +395,7 @@ done: char * _Py_SetLocaleFromEnv(int category) { + char *res; #ifdef __ANDROID__ const char *locale; const char **pvar; @@ -440,10 +442,12 @@ _Py_SetLocaleFromEnv(int category) } } #endif - return setlocale(category, utf8_locale); -#else /* __ANDROID__ */ - return setlocale(category, ""); -#endif /* __ANDROID__ */ + res = setlocale(category, utf8_locale); +#else /* !defined(__ANDROID__) */ + res = setlocale(category, ""); +#endif + _Py_ResetForceASCII(); + return res; } |