ruby-changes:50784
From: hsbt <ko1@a...>
Date: Wed, 28 Mar 2018 19:03:12 +0900 (JST)
Subject: [ruby-changes:50784] hsbt:r62990 (trunk): Ignore file separator from tmpfile/tmpdir name.
hsbt 2018-03-28 19:03:03 +0900 (Wed, 28 Mar 2018) New Revision: 62990 https://svn.ruby-lang.org/cgi-bin/viewvc.cgi?view=revision&revision=62990 Log: Ignore file separator from tmpfile/tmpdir name. From: SHIBATA Hiroshi <hsbt@r...> Modified files: trunk/lib/tmpdir.rb trunk/test/test_tempfile.rb trunk/test/test_tmpdir.rb Index: lib/tmpdir.rb =================================================================== --- lib/tmpdir.rb (revision 62989) +++ lib/tmpdir.rb (revision 62990) @@ -116,8 +116,10 @@ class Dir https://github.com/ruby/ruby/blob/trunk/lib/tmpdir.rb#L116 prefix, suffix = basename prefix = (String.try_convert(prefix) or raise ArgumentError, "unexpected prefix: #{prefix.inspect}") + prefix = prefix.delete("#{File::SEPARATOR}#{File::ALT_SEPARATOR}") suffix &&= (String.try_convert(suffix) or raise ArgumentError, "unexpected suffix: #{suffix.inspect}") + suffix &&= suffix.delete("#{File::SEPARATOR}#{File::ALT_SEPARATOR}") begin t = Time.now.strftime("%Y%m%d") path = "#{prefix}#{t}-#{$$}-#{rand(0x100000000).to_s(36)}"\ Index: test/test_tmpdir.rb =================================================================== --- test/test_tmpdir.rb (revision 62989) +++ test/test_tmpdir.rb (revision 62990) @@ -58,4 +58,21 @@ class TestTmpdir < Test::Unit::TestCase https://github.com/ruby/ruby/blob/trunk/test/test_tmpdir.rb#L58 assert_kind_of(String, d) } end + + TRAVERSAL_PATH = Array.new(Dir.pwd.split('/').count, '..').join('/') + Dir.pwd + '/' + TRAVERSAL_PATH.delete!(':') if /mswin|mingw/ =~ RUBY_PLATFORM + + def test_mktmpdir_traversal + expect = Dir.glob(TRAVERSAL_PATH + '*').count + Dir.mktmpdir(TRAVERSAL_PATH + 'foo') + actual = Dir.glob(TRAVERSAL_PATH + '*').count + assert_equal expect, actual + end + + def test_mktmpdir_traversal_array + expect = Dir.glob(TRAVERSAL_PATH + '*').count + Dir.mktmpdir([TRAVERSAL_PATH, 'foo']) + actual = Dir.glob(TRAVERSAL_PATH + '*').count + assert_equal expect, actual + end end Index: test/test_tempfile.rb =================================================================== --- test/test_tempfile.rb (revision 62989) +++ test/test_tempfile.rb (revision 62990) @@ -373,5 +373,31 @@ puts Tempfile.new('foo').path https://github.com/ruby/ruby/blob/trunk/test/test_tempfile.rb#L373 } assert_file.not_exist?(path) end -end + TRAVERSAL_PATH = Array.new(Dir.pwd.split('/').count, '..').join('/') + Dir.pwd + '/' + + def test_open_traversal_dir + expect = Dir.glob(TRAVERSAL_PATH + '*').count + t = Tempfile.open([TRAVERSAL_PATH, 'foo']) + actual = Dir.glob(TRAVERSAL_PATH + '*').count + assert_equal expect, actual + ensure + t.close! + end + + def test_new_traversal_dir + expect = Dir.glob(TRAVERSAL_PATH + '*').count + t = Tempfile.new(TRAVERSAL_PATH + 'foo') + actual = Dir.glob(TRAVERSAL_PATH + '*').count + assert_equal expect, actual + ensure + t.close! + end + + def test_create_traversal_dir + expect = Dir.glob(TRAVERSAL_PATH + '*').count + Tempfile.create(TRAVERSAL_PATH + 'foo') + actual = Dir.glob(TRAVERSAL_PATH + '*').count + assert_equal expect, actual + end +end -- ML: ruby-changes@q... Info: http://www.atdot.net/~ko1/quickml/