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

compile.c: fix load_from_binary

* compile.c (ibf_load_iseq_each): realpath may be nil.  follow up
  r59709.  [fix https://github.com/Shopify/bootsnap/issues/132]

From: nobu <nobu@b2dd03c8-39d4-4d8f-98ff-823fe69b080e>

git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@62772 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
This commit is contained in:
nobu 2018-03-16 02:27:50 +00:00
parent d3c5746bbb
commit ddf295a26f
2 changed files with 13 additions and 6 deletions

View file

@ -8773,7 +8773,15 @@ ibf_load_iseq_each(const struct ibf_load *load, rb_iseq_t *iseq, ibf_offset_t of
rb_raise(rb_eRuntimeError, "path object size mismatch");
}
path = rb_fstring(RARRAY_AREF(pathobj, 0));
realpath = rb_fstring(RARRAY_AREF(pathobj, 1));
realpath = RARRAY_AREF(pathobj, 1);
if (!NIL_P(realpath)) {
if (!RB_TYPE_P(realpath, T_STRING)) {
rb_raise(rb_eArgError, "unexpected realpath %"PRIxVALUE
"(%x), path=%+"PRIsVALUE,
realpath, TYPE(realpath), path);
}
realpath = rb_fstring(realpath);
}
}
else {
rb_raise(rb_eRuntimeError, "unexpected path object");

View file

@ -398,10 +398,9 @@ class TestISeq < Test::Unit::TestCase
def test_to_binary_with_objects
code = "[]"+100.times.map{|i|"<</#{i}/"}.join
bin = assert_nothing_raised {
RubyVM::InstructionSequence.compile(code).to_binary
}
# load_from_binary doesn't work now
assert_instance_of(String, bin)
iseq = RubyVM::InstructionSequence.compile(code)
bin = assert_nothing_raised {iseq.to_binary}
iseq2 = RubyVM::InstructionSequence.load_from_binary(bin)
assert_equal(iseq2.to_a, iseq.to_a)
end
end