From 2b7f6c22d868d23adb7cc8232464fe3f68b05b0d Mon Sep 17 00:00:00 2001
From: Tim Duesterhus <tim@bastelstu.be>
Date: Sat, 14 Mar 2020 13:07:05 +0100
Subject: [PATCH] CLEANUP: connection: Stop directly setting an ist's .ptr

Instead replace the complete `ist` by the value returned from `ist2`.

This was noticed during review of issue #549.
---
 src/connection.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/connection.c b/src/connection.c
index c1ec2664e..462dbf029 100644
--- a/src/connection.c
+++ b/src/connection.c
@@ -760,7 +760,7 @@ int conn_recv_proxy(struct connection *conn, int flag)
 
 				if (tlv.len > UNIQUEID_LEN)
 					goto bad_header;
-				conn->proxy_unique_id.ptr = pool_alloc(pool_head_uniqueid);
+				conn->proxy_unique_id = ist2(pool_alloc(pool_head_uniqueid), 0);
 				if (!isttest(conn->proxy_unique_id))
 					goto fail;
 				if (istcpy(&conn->proxy_unique_id, tlv, UNIQUEID_LEN) < 0) {