MINOR: mux-h2: remove a misleading and impossible test
In h2c_frt_handle_headers(), we test the stream for SS_ERROR just after
setting it to SS_OPEN, this makes no sense and creates confusion in the
error path. Remove this misleading test.
diff --git a/src/mux_h2.c b/src/mux_h2.c
index 3f8ae0d..d74d514 100644
--- a/src/mux_h2.c
+++ b/src/mux_h2.c
@@ -1917,15 +1917,9 @@
h2s->cs->flags |= CS_FL_REOS;
}
- if (h2s->st >= H2_SS_ERROR) {
- /* stream error : send RST_STREAM */
- h2c->st0 = H2_CS_FRAME_E;
- }
- else {
- /* update the max stream ID if the request is being processed */
- if (h2s->id > h2c->max_id)
- h2c->max_id = h2s->id;
- }
+ /* update the max stream ID if the request is being processed */
+ if (h2s->id > h2c->max_id)
+ h2c->max_id = h2s->id;
return h2s;