commit 2393c5b6a9c02325454cc43dbcb74838118768cd
author Willy Tarreau <w@1wt.eu> Wed Sep 05 15:24:56 2018 +0200
committer Willy Tarreau <w@1wt.eu> Wed Sep 05 20:01:23 2018 +0200
tree 9982f19bfd713abce6b5a08eacb7fc4eddccc5d4
parent 26ffa8544d70e096d26fd82199dc4837a5c169dc

MINOR: log: keep a copy of the backend connection early in sess_build_logline()

This way we can avoid dereferencing a possibly inexisting stream.

src/log.c

diff --git a/src/log.c b/src/log.c
index 0c5e1f9..f941c1b 100644
--- a/src/log.c
+++ b/src/log.c
@@ -1574,6 +1574,7 @@
 	struct proxy *be = s ? s->be : fe;
 	struct http_txn *txn = s ? s->txn : NULL;
 	const struct strm_logs *logs = s ? &s->logs : NULL;
+	const struct connection *be_conn = s ? cs_conn(objt_cs(s->si[1].end)) : NULL;
 	struct buffer chunk;
 	char *uri;
 	char *spc;
@@ -1711,9 +1712,8 @@
 				break;
 
 			case LOG_FMT_BACKENDIP:  // %bi
-				conn = cs_conn(objt_cs(s->si[1].end));
-				if (conn)
-					ret = lf_ip(tmplog, (struct sockaddr *)&conn->addr.from, dst + maxsize - tmplog, tmp);
+				if (be_conn)
+					ret = lf_ip(tmplog, (const struct sockaddr *)&be_conn->addr.from, dst + maxsize - tmplog, tmp);
 				else
 					ret = lf_text_len(tmplog, NULL, 0, dst + maxsize - tmplog, tmp);
 
@@ -1724,9 +1724,8 @@
 				break;
 
 			case LOG_FMT_BACKENDPORT:  // %bp
-				conn = cs_conn(objt_cs(s->si[1].end));
-				if (conn)
-					ret = lf_port(tmplog, (struct sockaddr *)&conn->addr.from, dst + maxsize - tmplog, tmp);
+				if (be_conn)
+					ret = lf_port(tmplog, (struct sockaddr *)&be_conn->addr.from, dst + maxsize - tmplog, tmp);
 				else
 					ret = lf_text_len(tmplog, NULL, 0, dst + maxsize - tmplog, tmp);
 
@@ -1737,9 +1736,8 @@
 				break;
 
 			case LOG_FMT_SERVERIP: // %si
-				conn = cs_conn(objt_cs(s->si[1].end));
-				if (conn)
-					ret = lf_ip(tmplog, (struct sockaddr *)&conn->addr.to, dst + maxsize - tmplog, tmp);
+				if (be_conn)
+					ret = lf_ip(tmplog, (struct sockaddr *)&be_conn->addr.to, dst + maxsize - tmplog, tmp);
 				else
 					ret = lf_text_len(tmplog, NULL, 0, dst + maxsize - tmplog, tmp);
 
@@ -1750,9 +1748,8 @@
 				break;
 
 			case LOG_FMT_SERVERPORT: // %sp
-				conn = cs_conn(objt_cs(s->si[1].end));
-				if (conn)
-					ret = lf_port(tmplog, (struct sockaddr *)&conn->addr.to, dst + maxsize - tmplog, tmp);
+				if (be_conn)
+					ret = lf_port(tmplog, (struct sockaddr *)&be_conn->addr.to, dst + maxsize - tmplog, tmp);
 				else
 					ret = lf_text_len(tmplog, NULL, 0, dst + maxsize - tmplog, tmp);