Probable use-after-free in VM assertion
The following Ruby program fails with VM assertions enabled on OpenBSD (code taken from
def foo return to_enum(__method__) unless block_given? raise yield 1 end enum = foo enum.next
This is due to the following assertion in
VM_ASSERT(ruby_current_vm_ptr == NULL || ruby_current_execution_context_ptr == NULL || rb_ec_thread_ptr(GET_EC()) == NULL || rb_ec_vm_ptr(GET_EC()) == ruby_current_vm_ptr);
Adding some debugging code,
0xdfdfdfdfdfdfdfdf. This is the memory pattern that OpenBSD free(3) writes to memory in order to detect use-after-free. So it is quite likely that this is operating on freed memory.
My guess as to what is happening here is that the enumerator fiber stack is freed, but this VM assertion is still accessing the memory. However, that's just a guess, and not a particularly educated one. I am not sure how to fix it.
Updated by jeremyevans0 (Jeremy Evans) 2 months ago
I think I've found a solution. If the thread for the EC has been killed, then don't check that the VM pointer matches, because the thread's memory (including the VM pointer) will have been freed. I've added a pull request that fixes this: https://github.com/ruby/ruby/pull/3443. This passes the bootstrap/basic tests on OpenBSD, which previously resulted in VM assertion failures without the change.
Updated by jeremyevans (Jeremy Evans) 2 months ago
- Status changed from Open to Closed