Commitbe0f17c

committed

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

PR-URL:nodejs/node#49400Reviewed-By: Matteo Collina <matteo.collina@gmail.com>Reviewed-By: Luigi Pinca <luigipinca@gmail.com>

1 parentc17ac8b commitbe0f17cCopy full SHA for be0f17c

File tree

-0

lines changed

-0

lines changed

Lines changed: 3 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -149,6 +149,7 @@ class JSStreamSocket extends Socket {`
`149`	`149`	`}`
`150`	`150`
`151`	`151`	`consthandle=this._handle;`
	`152`	`+assert(handle!==null);`
`152`	`153`
`153`	`154`	`process.nextTick(()=>{`
`154`	`155`	`// Ensure that write is dispatched asynchronously.`
`@@ -181,6 +182,8 @@ class JSStreamSocket extends Socket {`
`181`	`182`	`}`
`182`	`183`
`183`	`184`	`consthandle=this._handle;`
	`185`	`+assert(handle!==null);`
	`186`	`+`
`184`	`187`	`constself=this;`
`185`	`188`
`186`	`189`	`letpending=bufs.length;`

Comments

(0)