Skip to content

Commit 4ee9220

Browse files
reklatsmastersaddaleax
authored andcommitted
tty: avoid oob warning in TTYWrap::GetWindowSize()
PR-URL: #11454 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Anna Henningsen <[email protected]>
1 parent f37ab79 commit 4ee9220

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

lib/tty.js

+2-2
Original file line numberDiff line numberDiff line change
@@ -55,7 +55,7 @@ function WriteStream(fd) {
5555
// Ref: https://github.com/nodejs/node/pull/1771#issuecomment-119351671
5656
this._handle.setBlocking(process.env.NODE_TTY_UNSAFE_ASYNC !== '1');
5757

58-
var winSize = [];
58+
var winSize = new Array(2);
5959
var err = this._handle.getWindowSize(winSize);
6060
if (!err) {
6161
this.columns = winSize[0];
@@ -72,7 +72,7 @@ WriteStream.prototype.isTTY = true;
7272
WriteStream.prototype._refreshSize = function() {
7373
var oldCols = this.columns;
7474
var oldRows = this.rows;
75-
var winSize = [];
75+
var winSize = new Array(2);
7676
var err = this._handle.getWindowSize(winSize);
7777
if (err) {
7878
this.emit('error', errnoException(err, 'getWindowSize'));

0 commit comments

Comments
 (0)