Bug #15852
closedAPPLICATION_VERIFIER_LOCKS_LOCK_IN_FREED_HEAP on exiting ruby
Description
Ruby appears to be freeing some critical section in ruby_vm_destruct before deleting the critical section. Critical Section objects must be deleted with DeleteCriticalSection (https://docs.microsoft.com/en-us/windows/desktop/api/synchapi/nf-synchapi-deletecriticalsection) before freeing the owning memory.
I can't tell exactly what critical section is being freed - I'm new to ruby, don't know how the VM works, and haven't built it locally - so I can't help you guys there yet.
The clean stack trace is:
0:000> kc
# Call Site
00 vrfcore!VerifierStopMessageEx
01 vfbasics!AVrfpFreeMemLockChecks
02 vfbasics!AVrfpFreeMemNotify
03 vfbasics!AVrfpRtlFreeHeap
04 msvcrt!free
05 vfbasics!AVrfp_free
06 x64_msvcrt_ruby260!ruby_xfree
07 x64_msvcrt_ruby260!ruby_vm_destruct
08 x64_msvcrt_ruby260!ruby_cleanup
09 x64_msvcrt_ruby260!ruby_run_node
0a image00000000_00400000
0b image00000000_00400000
0c image00000000_00400000
0d KERNEL32!BaseThreadInitThunk
0e ntdll!RtlUserThreadStart
I'm running C:\Ruby26-x64\bin\ruby.exe -x "C:\Ruby26-x64\bin\irb.cmd"
, with Application Verifier lock verification enabled. Application Verifier is the Windows SDK dynamic analysis tool, and it can be enabled by anybody who's installed the SDK. Add ruby.exe to the list of applications in Application Verifier and then run irb from inside windbg. Leak analysis should be enabled by default.
I've attached the full debug log in Windbg.
Files
Updated by test35965@gmail.com (Alexander Riccio) over 5 years ago
Uhh, I don't know how to add maintainers, sorry!
Updated by test35965@gmail.com (Alexander Riccio) over 3 years ago
Still active as of Ruby 3.0.0p0 (x64-mingw32). I've done some more digging, and it appears to be freeing the Thread_Mutex in rb_call_inits (inits.c), aka the Init_Thread_Mutex function in thread.c.
Updated by test35965@gmail.com (Alexander Riccio) over 3 years ago
How do I assign? It looks like this should go to @ko1 (Koichi Sasada) (eval) or @nobu (Nobuyoshi Nakada) (mingw)
Updated by nobu (Nobuyoshi Nakada) over 3 years ago
vm->waitpid_lock
and vm->workqueue_lock
seem never deleted.
rb_native_mutex_destroy
on them are disabled by if (0)
in rb_vm_gvl_destroy()
, and that function is not called from anywhere too.
I have no idea why these calls are disabled.
Updated by nobu (Nobuyoshi Nakada) over 3 years ago
- Status changed from Open to Closed
Applied in changeset git|2a83650b0fd25719fb6c03bfec7bd895734d3ceb.
Destroy VM-wise locks before freeing [Bug #15852]
Updated by nobu (Nobuyoshi Nakada) over 3 years ago
- Backport changed from 2.4: UNKNOWN, 2.5: UNKNOWN, 2.6: UNKNOWN to 2.6: REQUIRED, 2.7: REQUIRED, 3.0: REQUIRED
vm->waitpid_lock
and vm->workqueue_lock
seem never deleted.
rb_native_mutex_destroy
on them are disabled by if (0)
in rb_vm_gvl_destroy()
, and that function is not called from anywhere too.
I have no idea why these calls are disabled.
Updated by naruse (Yui NARUSE) over 3 years ago
- Backport changed from 2.6: REQUIRED, 2.7: REQUIRED, 3.0: REQUIRED to 2.6: REQUIRED, 2.7: REQUIRED, 3.0: DONE
ruby_3_0 0074ea2d83230e10ab0cd769fff203c13d3c592f merged revision(s) 2a83650b0fd25719fb6c03bfec7bd895734d3ceb.
Updated by nagachika (Tomoyuki Chikanaga) over 3 years ago
- Backport changed from 2.6: REQUIRED, 2.7: REQUIRED, 3.0: DONE to 2.6: REQUIRED, 2.7: DONE, 3.0: DONE
ruby_2_7 30942fc4d89a04792e9eba7d3c4e5009c54e81da merged revision(s) 2a83650b0fd25719fb6c03bfec7bd895734d3ceb.
Updated by usa (Usaku NAKAMURA) over 3 years ago
- Backport changed from 2.6: REQUIRED, 2.7: DONE, 3.0: DONE to 2.6: DONE, 2.7: DONE, 3.0: DONE
backported into ruby_2_6 at r67930