From 298ff4fdb1513a944470a09619c5f494d142b3a2 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Micha=C3=ABl=20Zasso?= <targos@protonmail.com>
Date: Sun, 11 Aug 2024 08:32:13 +0200
Subject: [PATCH] tty: initialize winSize array with values

Assigning to a holey array in the native layer may crash if it has
getters that throw.

Refs: https://github.com/nodejs/node/issues/54186
PR-URL: https://github.com/nodejs/node/pull/54281
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
---
 lib/tty.js | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/lib/tty.js b/lib/tty.js
index 1a3fa5afe53..b1a2d3e7a9b 100644
--- a/lib/tty.js
+++ b/lib/tty.js
@@ -22,7 +22,6 @@
 'use strict';
 
 const {
-  Array,
   NumberIsInteger,
   ObjectSetPrototypeOf,
 } = primordials;
@@ -111,7 +110,7 @@ function WriteStream(fd) {
   // Ref: https://github.com/nodejs/node/pull/1771#issuecomment-119351671
   this._handle.setBlocking(true);
 
-  const winSize = new Array(2);
+  const winSize = [0, 0];
   const err = this._handle.getWindowSize(winSize);
   if (!err) {
     this.columns = winSize[0];
@@ -131,7 +130,7 @@ WriteStream.prototype.hasColors = hasColors;
 WriteStream.prototype._refreshSize = function() {
   const oldCols = this.columns;
   const oldRows = this.rows;
-  const winSize = new Array(2);
+  const winSize = [0, 0];
   const err = this._handle.getWindowSize(winSize);
   if (err) {
     this.emit('error', new ErrnoException(err, 'getWindowSize'));