BUG/MINOR: stream: don't emit a send-name-header in conn error or disconnect states

The test for the send-name-header field used to cover all states between
SI_ST_CON and SI_ST_CLO, which include SI_ST_CER and SI_ST_DIS. Trying to
send a header in these states makes no sense at all, so let's fix this.
This should have no visible impact so no backport is needed.

diff --git a/src/stream.c b/src/stream.c
index 0623ee8d4bcfa0407115c26822ae83e746f1ef84..bf1dd123e95ff629afb06e423893d30ebaf23198 100644 (file)
--- a/src/stream.c
+++ b/src/stream.c
@@ -2429,7 +2429,7 @@ redo:
 
                        /* Now we can add the server name to a header (if requested) */
                        /* check for HTTP mode and proxy server_name_hdr_name != NULL */
-                       if ((si_b->state >= SI_ST_CON) && (si_b->state < SI_ST_CLO) &&
+                       if ((si_b->state == SI_ST_CON || si_b->state == SI_ST_EST) &&
                            (s->be->server_id_hdr_name != NULL) &&
                            (s->be->mode == PR_MODE_HTTP) &&
                            objt_server(s->target)) {