From 628559765352624df9aff2ff7d73927e0df858de Mon Sep 17 00:00:00 2001 From: luislavena Date: Sat, 17 Nov 2012 12:46:17 +0000 Subject: [PATCH] Skip path expansion for all wildcard characters * win32/file.c (replace_to_long_name): skip expansion for all wildcard characters. [ruby-core:49451] [Bug #7374] * test/ruby/test_file_exhaustive.rb: add more assertions to test. git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@37700 b2dd03c8-39d4-4d8f-98ff-823fe69b080e --- ChangeLog | 8 ++++++++ test/ruby/test_file_exhaustive.rb | 4 +++- win32/file.c | 4 ++-- 3 files changed, 13 insertions(+), 3 deletions(-) diff --git a/ChangeLog b/ChangeLog index dbcf409161..ea49b01907 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,11 @@ +Sat Nov 17 21:45:12 Luis Lavena + + * win32/file.c (replace_to_long_name): skip expansion for all wildcard + characters. + [ruby-core:49451] [Bug #7374] + + * test/ruby/test_file_exhaustive.rb: add more assertions to test. + Sat Nov 17 12:14:50 2012 Aaron Patterson * ext/psych/lib/psych/visitors/yaml_tree.rb: use literal YAML style diff --git a/test/ruby/test_file_exhaustive.rb b/test/ruby/test_file_exhaustive.rb index 3fd8b3374e..360a1c7cce 100644 --- a/test/ruby/test_file_exhaustive.rb +++ b/test/ruby/test_file_exhaustive.rb @@ -662,9 +662,11 @@ class TestFileExhaustive < Test::Unit::TestCase assert_equal("#{DRIVE}/dir", File.expand_path("#{DRIVE}/./dir")) end - def test_expand_path_does_not_expand_wildcard + def test_expand_path_does_not_expand_wildcards assert_equal("#{DRIVE}/*", File.expand_path("./*", "#{DRIVE}/")) assert_equal("#{Dir.pwd}/*", File.expand_path("./*", Dir.pwd)) + assert_equal("#{DRIVE}/?", File.expand_path("./?", "#{DRIVE}/")) + assert_equal("#{Dir.pwd}/?", File.expand_path("./?", Dir.pwd)) end if DRIVE def test_expand_path_does_not_modify_the_string_argument diff --git a/win32/file.c b/win32/file.c index 1f033c646b..5550a5751d 100644 --- a/win32/file.c +++ b/win32/file.c @@ -276,8 +276,8 @@ replace_to_long_name(wchar_t **wfullpath, size_t size, int heap) return size; } - /* skip long name conversion if last character is a wildcard */ - if (pos[size - 1] == L'*') { + /* skip long name conversion if path contains wildcard characters */ + if (!wcspbrk(pos, "*?")) { return size; }