summaryrefslogtreecommitdiff
path: root/test/simple/test-timers-zero-timeout.js
diff options
context:
space:
mode:
authorYoshihiro Kikuchi <yknetg@gmail.com>2011-12-18 09:22:22 +0900
committerBen Noordhuis <info@bnoordhuis.nl>2011-12-18 01:50:01 +0100
commitf2f30286bf5e5332b1e788f3c745c5231e1b098b (patch)
treeffe445990b3f910e489ffa14080edfe98e613ea9 /test/simple/test-timers-zero-timeout.js
parent6df7bdd954efb817b50dafad7f90a1285c59c0c9 (diff)
downloadnode-new-f2f30286bf5e5332b1e788f3c745c5231e1b098b.tar.gz
timers: fix memory leak in setTimeout
Closing handle is leaked when setTimeout called with arguments which are 1. a callback 2. zero delay (i.e. setTimeout(function(){}, 0); )
Diffstat (limited to 'test/simple/test-timers-zero-timeout.js')
-rw-r--r--test/simple/test-timers-zero-timeout.js3
1 files changed, 3 insertions, 0 deletions
diff --git a/test/simple/test-timers-zero-timeout.js b/test/simple/test-timers-zero-timeout.js
index 79ecd9d267..d5e6df0786 100644
--- a/test/simple/test-timers-zero-timeout.js
+++ b/test/simple/test-timers-zero-timeout.js
@@ -27,6 +27,7 @@ var assert = require('assert');
var ncalled = 0;
setTimeout(f, 0, 'foo', 'bar', 'baz');
+ var timer = setTimeout(function(){}, 0);
function f(a, b, c) {
assert.equal(a, 'foo');
@@ -37,6 +38,8 @@ var assert = require('assert');
process.on('exit', function() {
assert.equal(ncalled, 1);
+ // timer should be already closed
+ assert.equal(timer.close(), -1);
});
})();