summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authornagachika <nagachika@b2dd03c8-39d4-4d8f-98ff-823fe69b080e>2018-03-21 15:57:56 +0000
committernagachika <nagachika@b2dd03c8-39d4-4d8f-98ff-823fe69b080e>2018-03-21 15:57:56 +0000
commit31ccc99d6295dfd6da71728592f7ea48cd809d60 (patch)
tree04bc59aef8466f5e085d3b7c31a18dc4835db52c
parentbb9a770cf34c9a6918fbc1ca6830c93ef3aa27a0 (diff)
downloadruby-31ccc99d6295dfd6da71728592f7ea48cd809d60.tar.gz
merge revision(s) 61862: [Backport #14368]
parse.y (new_regexp): Fix SEGV of `/#{"\u3042"}#{'{U+3044}'}/` in non UTF-8 Mixing other encoding string literals in one Regexp caused SEGV. This bug was found by CoverityScan. git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/branches/ruby_2_4@62880 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-rw-r--r--parse.y3
-rw-r--r--test/ruby/test_mixed_unicode_escapes.rb8
-rw-r--r--version.h2
3 files changed, 8 insertions, 5 deletions
diff --git a/parse.y b/parse.y
index 8e50087c26..ec2b056358 100644
--- a/parse.y
+++ b/parse.y
@@ -9141,8 +9141,7 @@ new_regexp_gen(struct parser_params *parser, NODE *node, int options)
if (reg_fragment_check(tail, options) && prev && !NIL_P(prev->nd_lit)) {
VALUE lit = prev == node ? prev->nd_lit : prev->nd_head->nd_lit;
if (!literal_concat0(parser, lit, tail)) {
- node = 0;
- break;
+ return NEW_NIL(); /* dummy node on error */
}
rb_str_resize(tail, 0);
prev->nd_next = list->nd_next;
diff --git a/test/ruby/test_mixed_unicode_escapes.rb b/test/ruby/test_mixed_unicode_escapes.rb
index 09240d8ab2..f0b4cc691f 100644
--- a/test/ruby/test_mixed_unicode_escapes.rb
+++ b/test/ruby/test_mixed_unicode_escapes.rb
@@ -13,14 +13,18 @@ class TestMixedUnicodeEscape < Test::Unit::TestCase
# 8-bit character escapes are okay.
assert_equal("B\xFF", "\u0042\xFF")
- # sjis mb chars mixed with Unicode shound not work
+ # sjis mb chars mixed with Unicode should not work
assert_raise(SyntaxError) { eval %q("\u1234")}
assert_raise(SyntaxError) { eval %q("\u{1234}")}
+ # also should not work for Regexp
+ assert_raise(SyntaxError) { eval %q(/#{"\u1234"}#{""}/)}
+ assert_raise(RegexpError) { eval %q(/\u{1234}#{nil}/)}
+ assert_raise(RegexpError) { eval %q(/#{nil}\u1234/)}
+
# String interpolation turns into an expression and we get
# a different kind of error, but we still can't mix these
assert_raise(Encoding::CompatibilityError) { eval %q("\u{1234}#{nil}")}
assert_raise(Encoding::CompatibilityError) { eval %q("#{nil}\u1234")}
-
end
end
diff --git a/version.h b/version.h
index 46c1b8e13f..c4bd04f96f 100644
--- a/version.h
+++ b/version.h
@@ -1,6 +1,6 @@
#define RUBY_VERSION "2.4.4"
#define RUBY_RELEASE_DATE "2018-03-22"
-#define RUBY_PATCHLEVEL 280
+#define RUBY_PATCHLEVEL 281
#define RUBY_RELEASE_YEAR 2018
#define RUBY_RELEASE_MONTH 3