From 0c3205a541bab01ce7f2b0a5f594d157d8154659 Mon Sep 17 00:00:00 2001 From: Willy Tarreau Date: Wed, 23 Mar 2022 11:11:31 +0100 Subject: [PATCH] BUILD: stream-int: avoid a build warning when DEBUG is empty When no DEBUG_STRICT is enabled, we get this build warning: src/stream_interface.c: In function 'stream_int_chk_snd_conn': src/stream_interface.c:1198:28: warning: unused variable 'conn' [-Wunused-variable] 1198 | struct connection *conn = cs_conn(cs); | ^~~~ This was the result of the simplification of the code in commit d1480cc8a ("BUG/MEDIUM: stream-int: do not rely on the connection error once established") which removed the last user of this variable outside of a BUG_ON(). If the patch above is backported, this one should be backported as well. --- src/stream_interface.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/src/stream_interface.c b/src/stream_interface.c index ae4678bd9..703fd37d0 100644 --- a/src/stream_interface.c +++ b/src/stream_interface.c @@ -1195,9 +1195,8 @@ static void stream_int_chk_snd_conn(struct stream_interface *si) { struct channel *oc = si_oc(si); struct conn_stream *cs = si->cs; - struct connection *conn = cs_conn(cs); - BUG_ON(!conn); + BUG_ON(!cs_conn(cs)); if (unlikely(!si_state_in(si->state, SI_SB_CON|SI_SB_RDY|SI_SB_EST) || (oc->flags & CF_SHUTW)))