From 70a311fdeb66ce80b562ccc4459c409cea64b74c Mon Sep 17 00:00:00 2001
From: Koichi Sasada <ko1@atdot.net>
Date: Sat, 12 Dec 2020 06:54:46 +0900
Subject: trap on non-main ractor

trap can accept blopck/Proc and it can violate Rator isolation,
so the Proc should be isolatable when trap is used on non-main ractor.
---
 signal.c | 6 ++++++
 1 file changed, 6 insertions(+)

(limited to 'signal.c')

diff --git a/signal.c b/signal.c
index 08de1bccdb..91521fe807 100644
--- a/signal.c
+++ b/signal.c
@@ -43,6 +43,7 @@
 #include "internal/thread.h"
 #include "ruby_atomic.h"
 #include "vm_core.h"
+#include "ractor_core.h"
 
 #ifdef NEED_RUBY_ATOMIC_OPS
 rb_atomic_t
@@ -1410,6 +1411,11 @@ sig_trap(int argc, VALUE *argv, VALUE _)
 	func = trap_handler(&cmd, sig);
     }
 
+    if (rb_obj_is_proc(cmd) &&
+        !rb_ractor_main_p() && !rb_ractor_shareable_p(cmd)) {
+        cmd = rb_proc_isolate(cmd);
+    }
+
     return trap(sig, func, cmd);
 }
 
-- 
cgit v1.2.1