IntenseFinance/neovim
3
0
Fork 0
mirror of https://github.com/neovim/neovim.git synced 2025-02-25 18:55:25 -06:00
Code Issues Packages Projects Releases Wiki Activity

env: use putenv_s for LC_ALL, LANG, etc. #11050

Browse Source 
Problem:  ":lang messages en_US.UTF-8" no longer overrides the language
          detected from the environment (at startup).
Solution: In os_setenv, special-case "LC_ALL", "LANG", et al. to use
          putenv_s instead of uv_os_setenv.

fixes #11045
This commit is contained in:
erw7 2019-09-22 16:02:28 +09:00 committed by Justin M. Keyes
parent c534ab8447
commit 028d76e5d5
1 changed files with 10 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
src/nvim/os/env.c
View File

@ -135,7 +135,16 @@ int os_setenv(const char *name, const char *value, int overwrite)
} }
#endif #endif
uv_mutex_lock(&mutex); uv_mutex_lock(&mutex);
int r = uv_os_setenv(name, value); int r;
#ifdef WIN32
// libintl uses getenv() for LC_ALL/LANG/etc., so we must use _putenv_s().
if (striequal(name, "LC_ALL") || striequal(name, "LANGUAGE")
|| striequal(name, "LANG") || striequal(name, "LC_MESSAGES")) {
r = _putenv_s(name, value); // NOLINT
assert(r == 0);
}
#endif
r = uv_os_setenv(name, value);
assert(r != UV_EINVAL); assert(r != UV_EINVAL);
// Destroy the old map item. Do this AFTER uv_os_setenv(), because `value` // Destroy the old map item. Do this AFTER uv_os_setenv(), because `value`
// could be a previous os_getenv() result. // could be a previous os_getenv() result.