kotovalexarian-likes-github/ruby--ruby
1
0
Fork 0
mirror of https://github.com/ruby/ruby.git synced 2022-11-09 12:17:21 -05:00
Code Releases Activity

Fix infinite loop at illegal sequence [Bug #17729]

Browse source 
As mblen returns -1 on failure, skip the first byte and try the
succeeding bytes in that case.

Close https://github.com/ruby/ruby/pull/4281
This commit is contained in:
Nobuyoshi Nakada 2021-03-18 18:48:56 +09:00
parent cc281bd7ac
commit f748b911c9
Notes: git 2021-03-19 07:16:16 +09:00 
Merged: https://github.com/ruby/ruby/pull/4284
2 changed files with 15 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
eval_intern.h
View file

@ -302,7 +302,16 @@ VALUE rb_ec_backtrace_location_ary(const rb_execution_context_t *ec, long lev, l
#ifndef CharNext /* defined as CharNext[AW] on Windows. */
# ifdef HAVE_MBLEN
# define CharNext(p) ((p) + mblen((p), RUBY_MBCHAR_MAXSIZE))
# define CharNext(p) rb_char_next(p)
static inline const char *
rb_char_next(const char *p)
{
if (p) {
int len = mblen(p, RUBY_MBCHAR_MAXSIZE);
p += len > 0 ? len : 1;
}
return p;
}
# else
# define CharNext(p) ((p) + 1)
# endif

5
test/ruby/test_rubyoptions.rb
View file

@ -1089,6 +1089,11 @@ class TestRubyOptions < Test::Unit::TestCase
end
end
def test_rubylib_invalid_encoding
env = {"RUBYLIB"=>"\xFF", "LOCALE"=>"en_US.UTF-8", "LC_ALL"=>"en_US.UTF-8"}
assert_ruby_status([env, "-e;"])
end
def test_null_script
skip "#{IO::NULL} is not a character device" unless File.chardev?(IO::NULL)
assert_in_out_err([IO::NULL], success: true)