MEDIUM: stream-int: always mark pending outgoing SI_ST_CON
Before the first send() attempt, we should be in SI_ST_CON, not
SI_ST_EST, since we have not yet attempted to send and we are
allowed to retry. This is particularly important with complex
outgoing muxes which can fail during the first send attempt (e.g.
failed stream ID allocation).
It only requires that sess_update_st_con_tcp() knows about this
possibility, as we must not forcefully close a reused connection
when facing an error in this case, this will be handled later.
This may be backported to 1.9 with care after some observation period.
diff --git a/include/proto/stream_interface.h b/include/proto/stream_interface.h
index cdc2245..5800c1e 100644
--- a/include/proto/stream_interface.h
+++ b/include/proto/stream_interface.h
@@ -491,10 +491,10 @@
si->state = SI_ST_CON;
}
else {
- /* reuse the existing connection */
-
- /* the connection is established */
- si->state = SI_ST_EST;
+ /* try to reuse the existing connection, it will be
+ * confirmed once we can send on it.
+ */
+ si->state = SI_ST_CON;
}
/* needs src ip/port for logging */