lib: connection - Turn connection_init_client_ip*() name parameter in…

…to an explicit hostname.

This way, the default connection name will be hostname:ip when hostname is not
NULL. This will be helpful once the connection API has TLS support.

src/lib-smtp/smtp-client-connection.c

@@ -1488,9 +1488,6 @@ static void
 smtp_client_connection_connect_next_ip(struct smtp_client_connection *conn)
 {
 	const struct ip_addr *ip, *my_ip = &conn->set.my_ip;
-	const char *conn_name = (conn->host_is_ip ? NULL :
-				 t_strdup_printf("%s:%u", conn->host,
-						 conn->port));
 
 	timeout_remove(&conn->to_connect);
 
@@ -1506,7 +1503,8 @@ smtp_client_connection_connect_next_ip(struct smtp_client_connection *conn)
 	}
 
 	connection_init_client_ip_from(conn->client->conn_list, &conn->conn,
-				       conn_name, ip, conn->port, my_ip);
+				       (conn->host_is_ip ? NULL : conn->host),
+				       ip, conn->port, my_ip);
 
 	smtp_client_connection_do_connect(conn);
 }

src/lib/connection.c

@@ -590,10 +590,16 @@ void connection_init_client_fd(struct connection_list *list,
 }
 
 void connection_init_client_ip_from(struct connection_list *list,
-				    struct connection *conn, const char *name,
+				    struct connection *conn,
+				    const char *hostname,
 				    const struct ip_addr *ip, in_port_t port,
 				    const struct ip_addr *my_ip)
 {
+	const char *name = NULL;
+
+	if (hostname != NULL)
+		name = t_strdup_printf("%s:%u", hostname, port);
+
 	i_assert(list->set.client);
 
 	conn->remote_ip = *ip;
@@ -615,10 +621,10 @@ void connection_init_client_ip_from(struct connection_list *list,
 }
 
 void connection_init_client_ip(struct connection_list *list,
-			       struct connection *conn, const char *name,
+			       struct connection *conn, const char *hostname,
 			       const struct ip_addr *ip, in_port_t port)
 {
-	connection_init_client_ip_from(list, conn, name, ip, port, NULL);
+	connection_init_client_ip_from(list, conn, hostname, ip, port, NULL);
 }
 
 void connection_init_client_unix(struct connection_list *list,

src/lib/connection.h

@@ -174,11 +174,12 @@ void connection_init_server_ip(struct connection_list *list,
 			       const struct ip_addr *remote_ip,
 			       in_port_t remote_port) ATTR_NULL(3, 6);
 void connection_init_client_ip(struct connection_list *list,
-			       struct connection *conn, const char *name,
+			       struct connection *conn, const char *hostname,
 			       const struct ip_addr *ip, in_port_t port)
 			       ATTR_NULL(3);
 void connection_init_client_ip_from(struct connection_list *list,
-				    struct connection *conn, const char *name,
+				    struct connection *conn,
+				    const char *hostname,
 				    const struct ip_addr *ip, in_port_t port,
 				    const struct ip_addr *my_ip) ATTR_NULL(3,6);
 void connection_init_client_unix(struct connection_list *list,