BUG/MINOR: mux-h1: Immediately report H1C errors from h1_snd_buf()

In case an H1 stream tries to send while on error occurred on its underlying
H1 connection, we must report the error. The way the stream-interface is
synchronously notified of the error. It seems to only be a problem on the
2.0. Probably because the scheduling has changed in upper versions. On the
2.0, it prevent the stream to be notified of errors, when for instance, a
payload is found in a response to a HEAD request. Not always though.

This patch must be backported as far as 2.0 because, on 2.0, it should fix the
issue #1101. There is no upstream ID for this commit because on the 2.4, this
fix already exists, it is part of non-backportable commit.

diff --git a/src/mux_h1.c b/src/mux_h1.c
index 62f555e..9312b33 100644 (file)
--- a/src/mux_h1.c
+++ b/src/mux_h1.c
@@ -2776,6 +2776,12 @@ static size_t h1_snd_buf(struct conn_stream *cs, struct buffer *buf, size_t coun
                return 0;
        }
 
+       if (h1c->flags & H1C_F_CS_ERROR) {
+               cs->flags |= CS_FL_ERROR;
+               TRACE_DEVEL("H1 connection is in error, leaving in error", H1_EV_STRM_SEND|H1_EV_H1C_ERR|H1_EV_H1S_ERR|H1_EV_STRM_ERR, h1c->conn, h1s);
+               return 0;
+       }
+
        /* Inherit some flags from the upper layer */
        h1c->flags &= ~(H1C_F_CO_MSG_MORE|H1C_F_CO_STREAMER);
        if (flags & CO_SFL_MSG_MORE)
@@ -2801,6 +2807,12 @@ static size_t h1_snd_buf(struct conn_stream *cs, struct buffer *buf, size_t coun
                if ((h1c->wait_event.events & SUB_RETRY_SEND) || !h1_send(h1c))
                        break;
        }
+
+       if (h1c->flags & H1C_F_CS_ERROR) {
+               TRACE_DEVEL("reporting error to the app-layer stream", H1_EV_STRM_SEND|H1_EV_H1S_ERR|H1_EV_STRM_ERR, h1c->conn, h1s);
+               cs->flags |= CS_FL_ERROR;
+       }
+
        h1_refresh_timeout(h1c);
        TRACE_LEAVE(H1_EV_STRM_SEND, h1c->conn, h1s, 0, (size_t[]){total});
        return total;