Hello,

small clean patch.
mutes coverity finding.

Ilya
From 4fdccb44933c2a91c7d6711bf821cc8b1d4c6d30 Mon Sep 17 00:00:00 2001
From: Ilya Shipitsin <[email protected]>
Date: Wed, 26 Apr 2023 21:05:12 +0200
Subject: [PATCH 1/2] CLEANUP: src/listener.c: remove redundant NULL check

fixes #2031

quoting Willy Tarreau:

"Originally the listeners were intended to work without a bind_conf
(e.g. for FTP processing) hence these tests, but over time the
bind_conf has become omnipresent"
---
 src/listener.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/listener.c b/src/listener.c
index d5390ed85..3da01df21 100644
--- a/src/listener.c
+++ b/src/listener.c
@@ -160,7 +160,7 @@ struct task *accept_queue_process(struct task *t, void 
*context, unsigned int st
                if (!(li->bind_conf->options & BC_O_UNLIMITED)) {
                        HA_ATOMIC_UPDATE_MAX(&global.sps_max,
                                             
update_freq_ctr(&global.sess_per_sec, 1));
-                       if (li->bind_conf && li->bind_conf->options & 
BC_O_USE_SSL) {
+                       if (li->bind_conf->options & BC_O_USE_SSL) {
                                HA_ATOMIC_UPDATE_MAX(&global.ssl_max,
                                                     
update_freq_ctr(&global.ssl_per_sec, 1));
                        }
-- 
2.39.2.windows.1

Reply via email to