[ruby/tmpdir] Reject empty parent path
https://github.com/ruby/tmpdir/commit/628c5bdc59
This commit is contained in:
parent
3231ac6008
commit
a79907ed5e
@ -148,7 +148,11 @@ class Dir
|
|||||||
# Generates and yields random names to create a temporary name
|
# Generates and yields random names to create a temporary name
|
||||||
def create(basename, tmpdir=nil, max_try: nil, **opts)
|
def create(basename, tmpdir=nil, max_try: nil, **opts)
|
||||||
origdir = tmpdir
|
origdir = tmpdir
|
||||||
tmpdir ||= tmpdir()
|
if tmpdir
|
||||||
|
raise ArgumentError, "empty parent path" if tmpdir.empty?
|
||||||
|
else
|
||||||
|
tmpdir = tmpdir()
|
||||||
|
end
|
||||||
n = nil
|
n = nil
|
||||||
prefix, suffix = basename
|
prefix, suffix = basename
|
||||||
prefix = (String.try_convert(prefix) or
|
prefix = (String.try_convert(prefix) or
|
||||||
|
@ -104,6 +104,12 @@ class TestTmpdir < Test::Unit::TestCase
|
|||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
|
def test_mktmpdir_not_empty_parent
|
||||||
|
assert_raise(ArgumentError) do
|
||||||
|
Dir.mktmpdir("foo", "")
|
||||||
|
end
|
||||||
|
end
|
||||||
|
|
||||||
def assert_mktmpdir_traversal
|
def assert_mktmpdir_traversal
|
||||||
Dir.mktmpdir do |target|
|
Dir.mktmpdir do |target|
|
||||||
target = target.chomp('/') + '/'
|
target = target.chomp('/') + '/'
|
||||||
|
Loading…
x
Reference in New Issue
Block a user