Raise a warning when listening for usafe signals #8601

Author: quibusus

lib/internal/process.js

@@ -191,6 +191,14 @@ function setupSignalHandlers() {
   process.on('newListener', function(type, listener) {
     if (isSignal(type) &&
         !signalWraps.hasOwnProperty(type)) {
+
+      if (['SIGBUS', 'SIGFPE', 'SIGSEGV', 'SIGILL'].indexOf(type) > -1) {
+        process.emitWarning(
+          'Listening to SIGBUS, SIGFPE, SIGSEGV, SIGILL signals ' +
+          'is not safe and the listener may be called in an ' +
+          'infinite loop');
+      }
+
       const Signal = process.binding('signal_wrap').Signal;
       const wrap = new Signal();
 

test/parallel/test-process-safe-signals.js

@@ -0,0 +1,5 @@
+'use strict';
+const common = require('../common');
+
+process.on('warning', common.fail);
+process.on('SIGINT', () => {});

test/parallel/test-process-unsafe-signals.js

@@ -0,0 +1,13 @@
+'use strict';
+const common = require('../common');
+const assert = require('assert');
+
+process.on('warning', common.mustCall((warning) => {
+  assert.strictEqual(warning.name, 'Warning');
+  assert.strictEqual(warning.message,
+                     'Listening to SIGBUS, SIGFPE, SIGSEGV, SIGILL signals ' +
+                     'is not safe and the listener may be called in an ' +
+                     'infinite loop');
+}));
+
+process.on('SIGBUS', () => {});