MEDIUM: stream-int: remove dangerous interval checks for stream-int states
The stream interface state checks involving ranges were replaced with
checks on a set of states, already revealing some issues. No issue was
fixed, all was replaced in a one-to-one mapping for easier control. Some
checks involving a strict difference were also replaced with fields to
be clearer. At this stage, the result must be strictly equivalent. A few
tests were also turned to their bit-field equivalent for better readability
or in preparation for upcoming changes.
The test performed in the SPOE filter was swapped so that the closed and
error states are evicted first and that the established vs conn state is
tested second.
diff --git a/src/flt_spoe.c b/src/flt_spoe.c
index 51d4f27..764e9ba 100644
--- a/src/flt_spoe.c
+++ b/src/flt_spoe.c
@@ -1341,15 +1341,18 @@
char *frame, *buf;
int ret;
- if (si->state <= SI_ST_CON) {
+ if (si_state_in(si->state, SI_SB_CER|SI_SB_DIS|SI_SB_CLO)) {
+ /* closed */
+ SPOE_APPCTX(appctx)->status_code = SPOE_FRM_ERR_IO;
+ goto exit;
+ }
+
+ if (!si_state_in(si->state, SI_SB_EST)) {
+ /* not connected yet */
si_rx_endp_more(si);
task_wakeup(si_strm(si)->task, TASK_WOKEN_MSG);
goto stop;
}
- if (si->state != SI_ST_EST) {
- SPOE_APPCTX(appctx)->status_code = SPOE_FRM_ERR_IO;
- goto exit;
- }
if (appctx->st1 == SPOE_APPCTX_ERR_TOUT) {
SPOE_PRINTF(stderr, "%d.%06d [SPOE/%-15s] %s: appctx=%p"