nodejs
diff --git a/Collapse file
‎src/node_http2.cc‎
Copy file name to clipboardExpand all lines: src/node_http2.cc
+2-1Lines changed: 2 additions & 1 deletion b/Collapse file
‎src/node_http2.cc‎
Copy file name to clipboardExpand all lines: src/node_http2.cc
+2-1Lines changed: 2 additions & 1 deletion
diff --git a/Collapse file
‎test/parallel/test-http2-close-while-writing.js‎
Copy file name to clipboard
+46Lines changed: 46 additions & 0 deletions b/Collapse file
‎test/parallel/test-http2-close-while-writing.js‎
Copy file name to clipboard
+46Lines changed: 46 additions & 0 deletions
@@ -92,7 +92,8 @@ Http2Scope::~Http2Scope() {
     return;
 
   session_->flags_ &= ~SESSION_STATE_HAS_SCOPE;
-  session_->MaybeScheduleWrite();
+  if (!(session_->flags_ & SESSION_STATE_WRITE_SCHEDULED))
+    session_->MaybeScheduleWrite();
 }
 
 // The Http2Options object is used during the construction of Http2Session
 
@@ -0,0 +1,46 @@
+'use strict';
+// https://github.com/nodejs/node/issues/33156
+const common = require('../common');
+const fixtures = require('../common/fixtures');
+
+if (!common.hasCrypto) {
+  common.skip('missing crypto');
+}
+
+const http2 = require('http2');
+
+const key = fixtures.readKey('agent8-key.pem', 'binary');
+const cert = fixtures.readKey('agent8-cert.pem', 'binary');
+const ca = fixtures.readKey('fake-startcom-root-cert.pem', 'binary');
+
+const server = http2.createSecureServer({
+  key,
+  cert,
+  maxSessionMemory: 1000
+});
+
+let client_stream;
+
+server.on('session', common.mustCall(function(session) {
+  session.on('stream', common.mustCall(function(stream) {
+    stream.resume();
+    stream.on('data', function() {
+      this.write(Buffer.alloc(1));
+      process.nextTick(() => client_stream.destroy());
+    });
+  }));
+}));
+
+server.listen(0, function() {
+  const client = http2.connect(`https://localhost:${server.address().port}`, {
+    ca,
+    maxSessionMemory: 1000
+  });
+  client_stream = client.request({ ':method': 'POST' });
+  client_stream.on('close', common.mustCall(() => {
+    client.close();
+    server.close();
+  }));
+  client_stream.resume();
+  client_stream.write(Buffer.alloc(64 * 1024));
+});
Original file line number	Diff line number	Diff line change
`@@ -92,7 +92,8 @@ Http2Scope::~Http2Scope() {`
`92`	`92`	`return;`
`93`	`93`
`94`	`94`	`session_->flags_ &= ~SESSION_STATE_HAS_SCOPE;`
`95`		`- session_->MaybeScheduleWrite();`
	`95`	`+ if (!(session_->flags_ & SESSION_STATE_WRITE_SCHEDULED))`
	`96`	`+ session_->MaybeScheduleWrite();`
`96`	`97`	`}`
`97`	`98`
`98`	`99`	`// The Http2Options object is used during the construction of Http2Session`