* file.c (rb_check_realpath_internal): infetct the result with
  arguments, no taint if none are tainted and cwd is not used.
  [ruby-core:83583] [Bug #14060]

git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@60599 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
This commit is contained in:
nobu 2017-11-01 02:22:10 +00:00
Родитель 76d4fa8d15
Коммит 2b43825fae
2 изменённых файлов: 21 добавлений и 1 удалений

2
file.c
Просмотреть файл

@ -4086,7 +4086,7 @@ rb_check_realpath_internal(VALUE basedir, VALUE path, enum rb_realpath_mode mode
}
}
OBJ_TAINT(resolved);
OBJ_INFECT(resolved, unresolved_path);
RB_GC_GUARD(unresolved_path);
RB_GC_GUARD(curdir);
return resolved;

Просмотреть файл

@ -283,6 +283,26 @@ class TestFile < Test::Unit::TestCase
}
end
def test_realpath_taintedness
Dir.mktmpdir('rubytest-realpath') {|tmpdir|
dir = File.realpath(tmpdir).untaint
File.write(File.join(dir, base = "test.file"), '')
base.taint
dir.taint
assert_predicate(File.realpath(base, dir), :tainted?)
base.untaint
dir.taint
assert_predicate(File.realpath(base, dir), :tainted?)
base.taint
dir.untaint
assert_predicate(File.realpath(base, dir), :tainted?)
base.untaint
dir.untaint
assert_not_predicate(File.realpath(base, dir), :tainted?)
assert_predicate(Dir.chdir(dir) {File.realpath(base)}, :tainted?)
}
end
def test_realdirpath
Dir.mktmpdir('rubytest-realdirpath') {|tmpdir|
realdir = File.realpath(tmpdir)