mirror of
https://github.com/ruby/ruby.git
synced 2022-11-09 12:17:21 -05:00
file.c: ASCII-compatible
* file.c (rb_file_join): check nul-byte only for strings, since FilePathStringValue() does it. [ruby-core:48012] [Bug #7168] git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@37212 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
This commit is contained in:
parent
7b4f0c0d1d
commit
a583f42062
3 changed files with 18 additions and 3 deletions
|
@ -1,4 +1,7 @@
|
|||
Tue Oct 16 10:54:03 2012 Nobuyoshi Nakada <nobu@ruby-lang.org>
|
||||
Tue Oct 16 11:27:04 2012 Nobuyoshi Nakada <nobu@ruby-lang.org>
|
||||
|
||||
* file.c (rb_file_join): check nul-byte only for strings, since
|
||||
FilePathStringValue() does it. [ruby-core:48012] [Bug #7168]
|
||||
|
||||
* file.c (rb_file_join): path names must be ASCII-compatible.
|
||||
[ruby-core:48012] [Bug #7168]
|
||||
|
|
4
file.c
4
file.c
|
@ -3941,6 +3941,7 @@ rb_file_join(VALUE ary, VALUE sep)
|
|||
tmp = RARRAY_PTR(ary)[i];
|
||||
switch (TYPE(tmp)) {
|
||||
case T_STRING:
|
||||
StringValueCStr(tmp);
|
||||
break;
|
||||
case T_ARRAY:
|
||||
if (ary == tmp) {
|
||||
|
@ -3957,8 +3958,7 @@ rb_file_join(VALUE ary, VALUE sep)
|
|||
default:
|
||||
FilePathStringValue(tmp);
|
||||
}
|
||||
name = StringValueCStr(result);
|
||||
len = RSTRING_LEN(result);
|
||||
RSTRING_GETMEM(result, name, len);
|
||||
if (i == 0) {
|
||||
rb_enc_copy(result, tmp);
|
||||
}
|
||||
|
|
|
@ -793,6 +793,18 @@ class TestFileExhaustive < Test::Unit::TestCase
|
|||
bug7168 = '[ruby-core:48012]'
|
||||
names = %w"a b".map {|s| s.encode(Encoding::UTF_16LE)}
|
||||
assert_raise(Encoding::CompatibilityError, bug7168) {File.join(*names)}
|
||||
assert_raise(Encoding::CompatibilityError, bug7168) {File.join(names)}
|
||||
|
||||
a = Object.new
|
||||
b = names[1]
|
||||
names = [a, "b"]
|
||||
a.singleton_class.class_eval do
|
||||
define_method(:to_path) do
|
||||
names[1] = b
|
||||
"a"
|
||||
end
|
||||
end
|
||||
assert_raise(Encoding::CompatibilityError, bug7168) {File.join(names)}
|
||||
end
|
||||
|
||||
def test_truncate
|
||||
|
|
Loading…
Reference in a new issue