From: Willy Tarreau <w@1wt.eu>
Date: Fri, 5 Oct 2018 08:22:27 +0000 (+0200)
Subject: MINOR: log: make sess_log() support sess=NULL
X-Git-Tag: v1.9-dev4~57
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=9fa267dada45ae2668d87651886dcf0f903d4012;p=thirdparty%2Fhaproxy.git

MINOR: log: make sess_log() support sess=NULL

At many places in muxes we'll have to add tests to check if the
connection is front or back before deciding to log. Instead let's
centralize this test in sess_log() to simply do nothing when sess=NULL.
---

diff --git a/src/log.c b/src/log.c
index 32308fa020..cd9ff01b99 100644
--- a/src/log.c
+++ b/src/log.c
@@ -2684,13 +2684,17 @@ void strm_log(struct stream *s)
  * cannot lead to the creation of a regular stream. Because of this the log
  * level is LOG_INFO or LOG_ERR depending on the "log-separate-error" setting
  * in the frontend. The caller must simply know that it should not call this
- * function to report unimportant events.
+ * function to report unimportant events. It is safe to call this function with
+ * sess==NULL (will not do anything).
  */
 void sess_log(struct session *sess)
 {
 	int size, level;
 	int sd_size = 0;
 
+	if (!sess)
+		return;
+
 	if (LIST_ISEMPTY(&sess->fe->logsrvs))
 		return;