From 338cd606f5e4ea84bbd396e9719ec4102b20973b Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Andr=C3=A9=20Cruz?= <andre@cabine.org>
Date: Fri, 28 Sep 2018 15:07:11 +0100
Subject: [PATCH] http2: don't send trailers on a closed connection

There is a race condition between onStreamCloseResponse(), which
removes the wantTrailers listener, and Http2Stream.close(), which
will invalidate the connection. IE, sendTrailers can be called on
a closed connection which would crash with a:
Error [ERR_HTTP2_INVALID_STREAM]: The stream has been destroyed
---
 lib/internal/http2/core.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lib/internal/http2/core.js b/lib/internal/http2/core.js
index 487bbe4f3b8a0f..a772fb277d7767 100644
--- a/lib/internal/http2/core.js
+++ b/lib/internal/http2/core.js
@@ -330,7 +330,7 @@ function tryClose(fd) {
 function onStreamTrailers() {
   const stream = this[kOwner];
   stream[kState].trailersReady = true;
-  if (stream.destroyed)
+  if (stream.destroyed || stream.closed)
     return;
   if (!stream.emit('wantTrailers')) {
     // There are no listeners, send empty trailing HEADERS frame and close.