From 79fd577ac148f78fe72154c26eae0edbd012bf46 Mon Sep 17 00:00:00 2001 From: Willy Tarreau Date: Thu, 23 Jan 2020 11:40:40 +0100 Subject: [PATCH] CLEANUP: backend: shut another false null-deref in back_handle_st_con() objt_conn() may return a NULL though here we don't have this situation anymore since the connection is always there, so let's simply switch to the unchecked __objt_conn(). This addresses issue #454. --- src/backend.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/backend.c b/src/backend.c index 859a00426..ed2c96943 100644 --- a/src/backend.c +++ b/src/backend.c @@ -1916,7 +1916,7 @@ void back_handle_st_con(struct stream *s) struct channel *req = &s->req; struct channel *rep = &s->res; struct conn_stream *srv_cs = objt_cs(si->end); - struct connection *conn = srv_cs ? srv_cs->conn : objt_conn(si->end); + struct connection *conn = srv_cs ? srv_cs->conn : __objt_conn(si->end); DBG_TRACE_ENTER(STRM_EV_STRM_PROC|STRM_EV_SI_ST, s);