1berry/nodejs
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

net: use asserts in JS Socket Stream to catch races in future

Browse Source 
PR-URL: https://github.com/nodejs/node/pull/49400
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
This commit is contained in:
Tim Perry 2023-08-25 14:16:35 +01:00 committed by Node.js GitHub Bot
parent dcf6670783
commit 47add7eb85
1 changed files with 3 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
lib/internal/js_stream_socket.js
View File

@ -149,6 +149,7 @@ class JSStreamSocket extends Socket {
} }
const handle = this._handle; const handle = this._handle;
assert(handle !== null);
process.nextTick(() => { process.nextTick(() => {
// Ensure that write is dispatched asynchronously. // Ensure that write is dispatched asynchronously.
@ -181,6 +182,8 @@ class JSStreamSocket extends Socket {
} }
const handle = this._handle; const handle = this._handle;
assert(handle !== null);
const self = this; const self = this;
let pending = bufs.length; let pending = bufs.length;