diff options
author | Yoshihiro Kikuchi <yknetg@gmail.com> | 2011-12-18 09:22:22 +0900 |
---|---|---|
committer | Ben Noordhuis <info@bnoordhuis.nl> | 2011-12-18 01:50:01 +0100 |
commit | f2f30286bf5e5332b1e788f3c745c5231e1b098b (patch) | |
tree | ffe445990b3f910e489ffa14080edfe98e613ea9 /test/simple/test-timers-zero-timeout.js | |
parent | 6df7bdd954efb817b50dafad7f90a1285c59c0c9 (diff) | |
download | node-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.js | 3 |
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); }); })(); |