diff options
author | usa <usa@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2014-11-28 07:30:21 +0000 |
---|---|---|
committer | usa <usa@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2014-11-28 07:30:21 +0000 |
commit | bcafeef2e753b4f02bea2e73bd5082bb789bea26 (patch) | |
tree | 8ab4e0e171474b77012b336b6d1b1a24a973b625 | |
parent | 7149859bc35cfe7497a1e98118f22a1628e8703d (diff) |
merge revision(s) 48360,48364: [Backport #10493]
* ext/etc/etc.c (etc_getlogin): set login name encoding properly.
[ruby-core:66163] [Bug #10493]
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/branches/ruby_2_0_0@48634 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-rw-r--r-- | ChangeLog | 5 | ||||
-rw-r--r-- | ext/etc/etc.c | 11 | ||||
-rw-r--r-- | test/etc/test_etc.rb | 4 | ||||
-rw-r--r-- | version.h | 2 | ||||
-rw-r--r-- | win32/win32.c | 30 |
5 files changed, 35 insertions, 17 deletions
@@ -1,3 +1,8 @@ +Fri Nov 28 16:28:07 2014 Nobuyoshi Nakada <nobu@ruby-lang.org> + + * ext/etc/etc.c (etc_getlogin): set login name encoding properly. + [ruby-core:66163] [Bug #10493] + Fri Nov 28 16:18:44 2014 Nobuyoshi Nakada <nobu@ruby-lang.org> * proc.c (method_proc): the receiver of binding from method should diff --git a/ext/etc/etc.c b/ext/etc/etc.c index dbf854404b..b9aceffcad 100644 --- a/ext/etc/etc.c +++ b/ext/etc/etc.c @@ -68,8 +68,15 @@ etc_getlogin(VALUE obj) login = getenv("USER"); #endif - if (login) - return rb_tainted_str_new2(login); + if (login) { +#ifdef _WIN32 + rb_encoding *extenc = rb_utf8_encoding(); +#else + rb_encoding *extenc = rb_locale_encoding(); +#endif + return rb_external_str_new_with_enc(login, strlen(login), extenc); + } + return Qnil; } diff --git a/test/etc/test_etc.rb b/test/etc/test_etc.rb index c105122af1..946be9bdf6 100644 --- a/test/etc/test_etc.rb +++ b/test/etc/test_etc.rb @@ -4,7 +4,9 @@ require "etc" class TestEtc < Test::Unit::TestCase def test_getlogin s = Etc.getlogin - assert(s.is_a?(String) || s == nil, "getlogin must return a String or nil") + return if s == nil + assert(s.is_a?(String), "getlogin must return a String or nil") + assert_predicate(s, :valid_encoding?, "login name should be a valid string") end def test_passwd @@ -1,6 +1,6 @@ #define RUBY_VERSION "2.0.0" #define RUBY_RELEASE_DATE "2014-11-28" -#define RUBY_PATCHLEVEL 603 +#define RUBY_PATCHLEVEL 604 #define RUBY_RELEASE_YEAR 2014 #define RUBY_RELEASE_MONTH 11 diff --git a/win32/win32.c b/win32/win32.c index 1dcad8693a..f85d4ec9d1 100644 --- a/win32/win32.c +++ b/win32/win32.c @@ -472,6 +472,16 @@ rb_w32_system_tmpdir(WCHAR *path, UINT len) return p - path + numberof(temp) - 1; } +static UINT filecp(void); +static WCHAR *mbstr_to_wstr(UINT, const char *, int, long *); +static char *wstr_to_mbstr(UINT, const WCHAR *, int, long *); +#define acp_to_wstr(str, plen) mbstr_to_wstr(CP_ACP, str, -1, plen) +#define wstr_to_acp(str, plen) wstr_to_mbstr(CP_ACP, str, -1, plen) +#define filecp_to_wstr(str, plen) mbstr_to_wstr(filecp(), str, -1, plen) +#define wstr_to_filecp(str, plen) wstr_to_mbstr(filecp(), str, -1, plen) +#define utf8_to_wstr(str, plen) mbstr_to_wstr(CP_UTF8, str, -1, plen) +#define wstr_to_utf8(str, plen) wstr_to_mbstr(CP_UTF8, str, -1, plen) + /* License: Ruby's */ static void init_env(void) @@ -518,11 +528,15 @@ init_env(void) if (!GetEnvironmentVariableW(L"USERNAME", env, numberof(env)) && !GetUserNameW(env, (len = numberof(env), &len))) { NTLoginName = "<Unknown>"; - return; } - set_env_val(L"USER"); + else { + set_env_val(L"USER"); + NTLoginName = wstr_to_mbstr(CP_UTF8, env, -1, NULL); + } + } + else { + NTLoginName = wstr_to_mbstr(CP_UTF8, env, -1, NULL); } - NTLoginName = strdup(rb_w32_getenv("USER")); if (!GetEnvironmentVariableW(TMPDIR, env, numberof(env)) && !GetEnvironmentVariableW(L"TMP", env, numberof(env)) && @@ -1165,16 +1179,6 @@ is_batch(const char *cmd) return 0; } -static UINT filecp(void); -static WCHAR *mbstr_to_wstr(UINT, const char *, int, long *); -static char *wstr_to_mbstr(UINT, const WCHAR *, int, long *); -#define acp_to_wstr(str, plen) mbstr_to_wstr(CP_ACP, str, -1, plen) -#define wstr_to_acp(str, plen) wstr_to_mbstr(CP_ACP, str, -1, plen) -#define filecp_to_wstr(str, plen) mbstr_to_wstr(filecp(), str, -1, plen) -#define wstr_to_filecp(str, plen) wstr_to_mbstr(filecp(), str, -1, plen) -#define utf8_to_wstr(str, plen) mbstr_to_wstr(CP_UTF8, str, -1, plen) -#define wstr_to_utf8(str, plen) wstr_to_mbstr(CP_UTF8, str, -1, plen) - /* License: Artistic or GPL */ rb_pid_t rb_w32_spawn(int mode, const char *cmd, const char *prog) |