timers: add v0.4 compatibility hack

If a timer callback throws and the user's uncaughtException handler ignores the
exception, other timers that expire on the current tick should still run.

If #2582 goes through, this hack should be removed.

Fixes #2631.

1 files changed, 15 insertions, 1 deletions

diff --git a/lib/timers.js b/lib/timers.js
index 7a57d34277..c5592bf0a2 100644
--- a/lib/timers.js
+++ b/lib/timers.js
@@ -81,7 +81,21 @@ function insert(item, msecs) {
         } else {
           L.remove(first);
           assert(first !== L.peek(list));
-          if (first._onTimeout) first._onTimeout();
+
+          if (!first._onTimeout) continue;
+
+          // v0.4 compatibility: if the timer callback throws and the user's
+          // uncaughtException handler ignores the exception, other timers that
+          // expire on this tick should still run. If #2582 goes through, this
+          // hack should be removed.
+          //
+          // https://github.com/joyent/node/issues/2631
+          try {
+            first._onTimeout();
+          } catch (e) {
+            if (!process.listeners('uncaughtException').length) throw e;
+            process.emit('uncaughtException', e);
+          }
         }
       }