From 9fa267dada45ae2668d87651886dcf0f903d4012 Mon Sep 17 00:00:00 2001 From: Willy Tarreau Date: Fri, 5 Oct 2018 10:22:27 +0200 Subject: [PATCH] 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. --- src/log.c | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/src/log.c b/src/log.c index 32308fa02..cd9ff01b9 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;