Rapid signal delivery via kill(2) causes SystemStackError
Running the following program with a trivial signal handler can crash with a SystemStackError if signals are delivered rapidly:
$ ruby -ve 'p Process.pid; trap "USR1" do 10 ** 100; end; sleep'
ruby 2.0.0dev (2012-02-11 trunk 34547) [x86_64-darwin11.2.0]
In a separate terminal:
ruby -e 'loop do Process.kill "USR1", 4504 end'
As each signal is delivered, ruby interrupts the current signal handler to perform the newly arrived one and quickly runs out of stack.
With an empty system handler the SystemStackError takes slightly longer to occur.
This also occurs with 1.9.3-p0:
$ ~/.multiruby/install/1.9.3-p0/bin/ruby -ve 'p Process.pid; trap "USR1" do 10 ** 100; end; sleep'
ruby 1.9.3p0 (2011-10-30 revision 33570) [x86_64-darwin11.2.0]
and with ruby-1.9.2-p290:
$ ~/.multiruby/install/1.9.2-p290/bin/ruby -ve 'p Process.pid; trap "USR1" do 10 ** 100; end; sleep'
ruby 1.9.2p290 (2011-07-09 revision 32553) [x86_64-darwin11.2.0]
ruby-1.8.7-p330 exits with a zero exit code:
$ ~/.multiruby/install/1.8.7-p330/bin/ruby -ve 'p Process.pid; trap "USR1" do 10 ** 100; end; sleep'
ruby 1.8.7 (2010-12-23 patchlevel 330) [i686-darwin11.2.0]
$ echo $?
Updated by kosaki (Motohiro KOSAKI) over 7 years ago
- Status changed from Assigned to Closed
- % Done changed from 0 to 100
This issue was solved with changeset r37861.
Eric, thank you for reporting this issue.
Your contribution to Ruby is greatly appreciated.
May Ruby be with you.
ruby_atomic.h (ATOMIC_CAS): new macro for compare-and-exchange.
vm_core.h (struct rb_thread_struct): add interrupt_mask member.
thread.c (thread_create_core, Init_Thread): initialize
vm_core.h (RUBY_VM_INTERRUPTED_ANY): new macro for avoiding
vm_core.h (RUBY_VM_INTERRUPTED, RUBY_VM_INTERRUPTED): check
thread.c (set_unblock_function, rb_thread_schedule): replace
th->interrupt_flag with RUBY_VM_INTERRUPTED_ANY()
signal.c (signal_exec): set up thread->interrupt_mask for
preventing recursive trap handler.
vm_core.h (RUBY_VM_CHECK_INTS, RUBY_VM_CHECK_INTS_BLOCKING): ditto.
don't process interrupt if it is masked.
[Bug #6009] [ruby-core:42524]