git.99rst.org Git - openwrt-packages.git/blob

   1 commit 105abe2f7a2e518afda9eb3bda5cceb60f6fd1b2
   2 Author: Olivier Houchard <cognet@ci0.org>
   3 Date:   Sun Oct 21 03:01:20 2018 +0200
   4
   5     BUG/MEDIUM: h2: Close connection if no stream is left an GOAWAY was sent.
   6
   7     When we're closing a stream, is there's no stream left and a goaway was sent,
   8     close the connection, there's no reason to keep it open.
   9
  10     [wt: it's likely that this is needed in 1.8 as well, though it's unclear
  11      how to trigger this issue, some tests are needed]
  12
  13     (cherry picked from commit 52b946686c28891a4359e9361676dc62af4fffad)
  14     Signed-off-by: Willy Tarreau <w@1wt.eu>
  15
  16 diff --git a/src/mux_h2.c b/src/mux_h2.c
  17 index b1b039fe..6881302b 100644
  18 --- a/src/mux_h2.c
  19 +++ b/src/mux_h2.c
  20 @@ -2540,7 +2540,7 @@ static void h2_detach(struct conn_stream *cs)
  21         if (eb_is_empty(&h2c->streams_by_id) &&     /* don't close if streams exist */
  22             ((h2c->conn->flags & CO_FL_ERROR) ||    /* errors close immediately */
  23              (h2c->st0 >= H2_CS_ERROR && !h2c->task) || /* a timeout stroke earlier */
  24 -            (h2c->flags & H2_CF_GOAWAY_FAILED) ||
  25 +            (h2c->flags & (H2_CF_GOAWAY_FAILED | H2_CF_GOAWAY_SENT)) ||
  26              (!h2c->mbuf->o &&  /* mux buffer empty, also process clean events below */
  27               (conn_xprt_read0_pending(h2c->conn) ||
  28                (h2c->last_sid >= 0 && h2c->max_id >= h2c->last_sid))))) {