From fcaee4bd64eb06ec4d0afd4e7e32ccf57acb5dc8 Mon Sep 17 00:00:00 2001 From: usa Date: Thu, 15 Mar 2012 03:57:02 +0000 Subject: * win32/win32.c, include/ruby/win32.h (rb_w32_ugetenv): new API to accept and to return UTF-8 strings. * win32/win32.c (rb_w32_getenv): follow above change. * win32/win32.c (rb_w32_get_environ): returns UTF-8 environment area. * hash.c (env_str_new, rb_f_getenv, env_fetch): follow above changes. [Bug #5570] [ruby-core:40737] git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@35030 b2dd03c8-39d4-4d8f-98ff-823fe69b080e --- win32/win32.c | 67 ++++++++++++++++++++++++++++++++++++++++++++++++++++------- 1 file changed, 59 insertions(+), 8 deletions(-) (limited to 'win32/win32.c') diff --git a/win32/win32.c b/win32/win32.c index 744c61489e..37b3943036 100644 --- a/win32/win32.c +++ b/win32/win32.c @@ -630,6 +630,7 @@ static CRITICAL_SECTION select_mutex; static int NtSocketsInitialized = 0; static st_table *socklist = NULL; static char *envarea; +static char *uenvarea; /* License: Ruby's */ static void @@ -648,6 +649,10 @@ exit_handler(void) FreeEnvironmentStrings(envarea); envarea = NULL; } + if (uenvarea) { + free(uenvarea); + uenvarea = NULL; + } } /* License: Artistic or GPL */ @@ -4234,6 +4239,44 @@ wait(int *status) return waitpid(-1, status, 0); } +/* License: Ruby's */ +char * +rb_w32_ugetenv(const char *name) +{ + WCHAR *wenvarea, *wenv; + int len = strlen(name); + char *env; + int wlen; + + if (len == 0) return NULL; + + if (uenvarea) { + free(uenvarea); + uenvarea = NULL; + } + if (envarea) { + FreeEnvironmentStrings(envarea); + envarea = NULL; + } + wenvarea = GetEnvironmentStringsW(); + if (!wenvarea) { + map_errno(GetLastError()); + return NULL; + } + for (wenv = wenvarea, wlen = 1; *wenv; wenv += lstrlenW(wenv) + 1) + wlen += lstrlenW(wenv) + 1; + uenvarea = wstr_to_mbstr(CP_UTF8, wenvarea, wlen, NULL); + FreeEnvironmentStringsW(wenvarea); + if (!uenvarea) + return NULL; + + for (env = uenvarea; *env; env += strlen(env) + 1) + if (strncasecmp(env, name, len) == 0 && *(env + len) == '=') + return env + len + 1; + + return NULL; +} + /* License: Ruby's */ char * rb_w32_getenv(const char *name) @@ -4242,7 +4285,14 @@ rb_w32_getenv(const char *name) char *env; if (len == 0) return NULL; - if (envarea) FreeEnvironmentStrings(envarea); + if (uenvarea) { + free(uenvarea); + uenvarea = NULL; + } + if (envarea) { + FreeEnvironmentStrings(envarea); + envarea = NULL; + } envarea = GetEnvironmentStrings(); if (!envarea) { map_errno(GetLastError()); @@ -5011,7 +5061,7 @@ rb_w32_asynchronize(asynchronous_func_t func, uintptr_t self, char ** rb_w32_get_environ(void) { - char *envtop, *env; + WCHAR *envtop, *env; char **myenvtop, **myenv; int num; @@ -5022,23 +5072,24 @@ rb_w32_get_environ(void) * CygWin deals these values by changing first `=' to '!'. But we don't * use such trick and follow cmd.exe's way that just doesn't show these * values. - * (U.N. 2001-11-15) + * + * This function returns UTF-8 strings. */ - envtop = GetEnvironmentStrings(); - for (env = envtop, num = 0; *env; env += strlen(env) + 1) + envtop = GetEnvironmentStringsW(); + for (env = envtop, num = 0; *env; env += lstrlenW(env) + 1) if (*env != '=') num++; myenvtop = (char **)malloc(sizeof(char *) * (num + 1)); - for (env = envtop, myenv = myenvtop; *env; env += strlen(env) + 1) { + for (env = envtop, myenv = myenvtop; *env; env += lstrlenW(env) + 1) { if (*env != '=') { - if (!(*myenv = strdup(env))) { + if (!(*myenv = wstr_to_utf8(env, NULL))) { break; } myenv++; } } *myenv = NULL; - FreeEnvironmentStrings(envtop); + FreeEnvironmentStringsW(envtop); return myenvtop; } -- cgit v1.2.3