Project

General

Profile

Actions

Bug #13960

closed

ARGF.inplace_mode= が SHARABLE_MIDDLE_SUBSTRING=1 を考慮していない

Added by tommy (Masahiro Tomita) over 6 years ago. Updated over 6 years ago.

Status:
Closed
Assignee:
-
Target version:
-
ruby -v:
ruby 2.5.0dev (2017-10-01 trunk 60085) [x86_64-linux]
[ruby-dev:50272]

Description

SHARABLE_MIDDLE_SUBSTRING=1 でコンパイルしたRubyで、ARGF.inplace_mode= がNUL終端まで文字列を使用してしまいます。

% echo hoge > hoge
% ruby -e 'ARGF.inplace_mode=(".abcdefghijklmnopqrstuvwxyz"*2)[0,27]; ARGF.each_line{|line| print line.sub("hoge","fuga")}' hoge
% ls hoge*   
hoge  hoge.abcdefghijklmnopqrstuvwxyz.abcdefghijklmnopqrstuvwxyz

次のパッチで直ると思います。

diff --git a/io.c b/io.c
index 1254e92363..ed4a562d8b 100644
--- a/io.c
+++ b/io.c
@@ -12224,7 +12224,7 @@ argf_inplace_mode_set(VALUE argf, VALUE val)
 	ARGF.inplace = 0;
     }
     else {
-	StringValue(val);
+	StringValueCStr(val);
 	if (ARGF.inplace) free(ARGF.inplace);
 	ARGF.inplace = 0;
 	ARGF.inplace = strdup(RSTRING_PTR(val));
Actions #1

Updated by nobu (Nobuyoshi Nakada) over 6 years ago

  • Status changed from Open to Closed

Applied in changeset trunk|r60097.


io.c: check null char

  • io.c (ruby_set_inplace_mode): check if null is contained. based
    on the patch by tommy (Masahiro Tomita) in [ruby-dev:50272].
    [Bug #13960]
Actions

Also available in: Atom PDF

Like0
Like0