403d7ee7d1
Commit 934bfe23a1 had introduced a
regression where node would crash trying to access a null unref timer if
a given unref timer's callback would remove other unref timers set to
fire in the future.
More generally, it makes the unrefTimeout function more solid by not
mutating the unrefList while traversing it.
Fixes: https://github.com/joyent/node/issues/8897
Conflicts:
lib/timers.js
Fixes: https://github.com/nodejs/node-convergence-archive/issues/23
Ref: https://github.com/nodejs/node/issues/268
PR-URL: https://github.com/nodejs/node/pull/2540
Reviewed-By: bnoordhuis - Ben Noordhuis <info@bnoordhuis.nl>
33 lines
813 B
JavaScript
33 lines
813 B
JavaScript
'use strict';
|
|
|
|
/*
|
|
* This test is a regression test for joyent/node#8897.
|
|
*
|
|
* It tests some private implementation details that should not be
|
|
* considered public interface.
|
|
*/
|
|
const common = require('../common');
|
|
const assert = require('assert');
|
|
const timers = require('timers');
|
|
|
|
const foo = {
|
|
_onTimeout: assert.fail
|
|
};
|
|
|
|
const bar = {
|
|
_onTimeout: common.mustCall(function() {
|
|
timers.unenroll(foo);
|
|
})
|
|
};
|
|
|
|
// We use timers with expiration times that are sufficiently apart to make
|
|
// sure that they're not fired at the same time on platforms where the timer
|
|
// resolution is a bit coarse (e.g Windows with a default resolution of ~15ms).
|
|
timers.enroll(bar, 1);
|
|
timers._unrefActive(bar);
|
|
|
|
timers.enroll(foo, 50);
|
|
timers._unrefActive(foo);
|
|
|
|
// Keep the process open.
|
|
setTimeout(function() {}, 100);
|