Skip to content
This repository was archived by the owner on Apr 22, 2023. It is now read-only.

Commit 1953886

Browse files
committed
test: fs.createReadStream().destroy() fd leak
PR-URL: nodejs/node#56 Reviewed-By: Rod Vagg <[email protected]> See PR for long discussion
1 parent 497fd72 commit 1953886

File tree

1 file changed

+66
-0
lines changed

1 file changed

+66
-0
lines changed
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,66 @@
1+
// Copyright io.js contributors.
2+
//
3+
// Permission is hereby granted, free of charge, to any person obtaining a
4+
// copy of this software and associated documentation files (the
5+
// "Software"), to deal in the Software without restriction, including
6+
// without limitation the rights to use, copy, modify, merge, publish,
7+
// distribute, sublicense, and/or sell copies of the Software, and to permit
8+
// persons to whom the Software is furnished to do so, subject to the
9+
// following conditions:
10+
//
11+
// The above copyright notice and this permission notice shall be included
12+
// in all copies or substantial portions of the Software.
13+
//
14+
// THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS
15+
// OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
16+
// MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN
17+
// NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM,
18+
// DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR
19+
// OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE
20+
// USE OR OTHER DEALINGS IN THE SOFTWARE.
21+
22+
var common = require('../common');
23+
var assert = require('assert');
24+
var fs = require('fs');
25+
var path = require('path');
26+
27+
var openCount = 0;
28+
var _fsopen = fs.open;
29+
var _fsclose = fs.close;
30+
31+
var loopCount = 50;
32+
var emptyTxt = path.join(common.fixturesDir, 'empty.txt');
33+
34+
fs.open = function() {
35+
openCount++;
36+
return _fsopen.apply(null, arguments);
37+
}
38+
39+
fs.close = function() {
40+
openCount--;
41+
return _fsclose.apply(null, arguments);
42+
}
43+
44+
function testLeak(endFn, callback) {
45+
console.log('testing for leaks from fs.createReadStream().%s()...', endFn);
46+
47+
var i = 0;
48+
49+
setInterval(function() {
50+
var s = fs.createReadStream(emptyTxt);
51+
s[endFn]();
52+
53+
if (++i === loopCount) {
54+
clearTimeout(this);
55+
setTimeout(function() {
56+
assert.equal(0, openCount, 'no leaked file descriptors using ' + endFn + '() (got ' + openCount + ')');
57+
openCount = 0;
58+
callback && setTimeout(callback, 100);
59+
}, 100);
60+
}
61+
}, 2);
62+
}
63+
64+
testLeak('close', function() {
65+
testLeak('destroy');
66+
});

0 commit comments

Comments
 (0)