Skip to content

Commit ef63af6

Browse files
reklatsmastersMylesBorins
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 6a45ac0 commit ef63af6

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

lib/tty.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -66,7 +66,7 @@ function WriteStream(fd) {
6666
// Ref: https://github.com/nodejs/node/pull/1771#issuecomment-119351671
6767
if (process.platform === 'darwin') this._handle.setBlocking(true);
6868

69-
var winSize = [];
69+
var winSize = new Array(2);
7070
var err = this._handle.getWindowSize(winSize);
7171
if (!err) {
7272
this.columns = winSize[0];
@@ -83,7 +83,7 @@ WriteStream.prototype.isTTY = true;
8383
WriteStream.prototype._refreshSize = function() {
8484
var oldCols = this.columns;
8585
var oldRows = this.rows;
86-
var winSize = [];
86+
var winSize = new Array(2);
8787
var err = this._handle.getWindowSize(winSize);
8888
if (err) {
8989
this.emit('error', errnoException(err, 'getWindowSize'));

0 commit comments

Comments
 (0)