quic: fixes undefined handle in QuicStream kInspect

martenrichter · richardlau · commit 2699fe470640 · 2026-07-03T16:05:40.000Z
If the #handle is undefined and not QuicStream, while kInspect is called `DataViewPrototypeGetByteLength` will throw. Signed-off-by: Marten Richter <marten.richter@freenet.de> PR-URL: #64170 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Gürgün Dayıoğlu <hey@gurgun.day> Reviewed-By: Tim Perry <pimterry@gmail.com>
diff --git a/lib/internal/quic/state.js b/lib/internal/quic/state.js
@@ -933,7 +933,8 @@ class QuicStreamState {
   }
 
   [kInspect](depth, options) {
-    if (DataViewPrototypeGetByteLength(this.#handle) === 0) {
+    if (this.#handle === undefined ||
+      DataViewPrototypeGetByteLength(this.#handle) === 0) {
       return 'QuicStreamState { <Closed> }';
     }
 

Original file line number	Diff line number	Diff line change
`@@ -933,7 +933,8 @@ class QuicStreamState {`
`933`	`933`	`}`
`934`	`934`
`935`	`935`	`[kInspect](depth, options) {`
`936`		`- if (DataViewPrototypeGetByteLength(this.#handle) === 0) {`
	`936`	`+ if (this.#handle === undefined \|\|`
	`937`	`+ DataViewPrototypeGetByteLength(this.#handle) === 0) {`
`937`	`938`	`return 'QuicStreamState { <Closed> }';`
`938`	`939`	`}`
`939`	`940`