BUG/MEDIUM: connection: Don't crush context pointer location if it is a CS

The issue only concerns the backend connection. The conn-stream is now owned
by the stream and persists during all the stream life. Thus we must not
crush it when the backend connection is released.

It is 2.6-specific. No backport is needed.
This commit is contained in:
Christopher Faulet 2022-04-15 10:57:09 +02:00
parent cef08c20c7
commit 2bb5edcf19

View File

@ -501,13 +501,6 @@ void conn_free(struct connection *conn)
pool_free(pool_head_conn_hash_node, conn->hash_node);
conn->hash_node = NULL;
/* By convention we always place a NULL where the ctx points to if the
* mux is null. It may have been used to store the connection as a
* conn-stream's end point for example.
*/
if (conn->ctx != NULL && conn->mux == NULL)
*(void **)conn->ctx = NULL;
conn_force_unsubscribe(conn);
pool_free(pool_head_connection, conn);
}