diff options
author | usa <usa@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2010-09-30 07:12:49 +0000 |
---|---|---|
committer | usa <usa@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2010-09-30 07:12:49 +0000 |
commit | 1fcb7a5ee0f346f755735d349a405eeccf81fc40 (patch) | |
tree | 5d9ba28eb1a9390c531b5b4109c71a259837e198 | |
parent | 85bcd2587794dbe653523847bc01cab4948be2d4 (diff) | |
download | ruby-1fcb7a5ee0f346f755735d349a405eeccf81fc40.tar.gz |
* win32/win32.c (rb_w32_getenv): should return NULL if specified name
is empty. a patch from Heesob Park at [ruby-core:32650]
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@29378 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-rw-r--r-- | ChangeLog | 5 | ||||
-rw-r--r-- | test/ruby/test_env.rb | 1 | ||||
-rw-r--r-- | win32/win32.c | 4 |
3 files changed, 8 insertions, 2 deletions
@@ -1,3 +1,8 @@ +Thu Sep 30 16:11:08 2010 NAKAMURA Usaku <usa@ruby-lang.org> + + * win32/win32.c (rb_w32_getenv): should return NULL if specified name + is empty. a patch from Heesob Park at [ruby-core:32650] + Thu Sep 30 10:55:38 2010 NAKAMURA Usaku <usa@ruby-lang.org> * hash.c (ruby_setenv): workaround for old Windows. a patch from diff --git a/test/ruby/test_env.rb b/test/ruby/test_env.rb index 97bf8fa476..681b21d614 100644 --- a/test/ruby/test_env.rb +++ b/test/ruby/test_env.rb @@ -96,6 +96,7 @@ class TestEnv < Test::Unit::TestCase assert_raise(ArgumentError) { ENV["foo\0bar"] } ENV[PATH_ENV] = "" assert_equal("", ENV[PATH_ENV]) + assert_nil(ENV[""]) end def test_fetch diff --git a/win32/win32.c b/win32/win32.c index 631b3e24e1..2dd9d0e023 100644 --- a/win32/win32.c +++ b/win32/win32.c @@ -3900,8 +3900,8 @@ rb_w32_getenv(const char *name) int len = strlen(name); char *env; - if (envarea) - FreeEnvironmentStrings(envarea); + if (len == 0) return NULL; + if (envarea) FreeEnvironmentStrings(envarea); envarea = GetEnvironmentStrings(); if (!envarea) { map_errno(GetLastError()); |