X-Git-Url: https://git.liburcu.org/?a=blobdiff_plain;f=src%2Fcommon%2Furi.c;fp=src%2Fcommon%2Furi.c;h=4ad053b84da097c50622b13972fc31ddb5b7c06b;hb=84f4ce0977fcb42ca4a0aeb9b4feb7b8297d3b04;hp=408b75cd73c7e9808a47c30196372a2ffbe8dde6;hpb=b2f26ff8607f18abba98718fb6a114ea04142975;p=lttng-tools.git

diff --git a/src/common/uri.c b/src/common/uri.c
index 408b75cd7..4ad053b84 100644
--- a/src/common/uri.c
+++ b/src/common/uri.c
@@ -29,6 +29,9 @@
 
 #include "uri.h"
 
+#define LOOPBACK_ADDR_IPV4 "127.0.0.1"
+#define LOOPBACK_ADDR_IPV6 "::1"
+
 enum uri_proto_code {
 	P_NET, P_NET6, P_FILE, P_TCP, P_TCP6,
 };
@@ -116,14 +119,43 @@ static int set_ip_address(const char *addr, int af, char *dst, size_t size)
 	if (ret < 1) {
 		/* We consider the dst to be an hostname or an invalid IP char */
 		record = gethostbyname2(addr, af);
-		if (record == NULL) {
+		if (record) {
+			/* Translate IP to string */
+			if (!inet_ntop(af, record->h_addr_list[0], dst, size)) {
+				PERROR("inet_ntop");
+				goto error;
+			}
+		} else if (!strcmp(addr, "localhost") &&
+				(af == AF_INET || af == AF_INET6)) {
+			/*
+			 * Some systems may not have "localhost" defined in
+			 * accordance with IETF RFC 6761. According to this RFC,
+			 * applications may recognize "localhost" names as
+			 * special and resolve to the appropriate loopback
+			 * address.
+			 *
+			 * We choose to use the system name resolution API first
+			 * to honor its network configuration. If this fails, we
+			 * resolve to the appropriate loopback address. This is
+			 * done to accomodate systems which may want to start
+			 * tracing before their network configured.
+			 */
+			const char *loopback_addr = af == AF_INET ?
+					LOOPBACK_ADDR_IPV4 : LOOPBACK_ADDR_IPV6;
+			const size_t loopback_addr_len = af == AF_INET ?
+					sizeof(LOOPBACK_ADDR_IPV4) :
+					sizeof(LOOPBACK_ADDR_IPV6);
+
+			DBG2("Could not resolve localhost address, using fallback");
+			if (loopback_addr_len > size) {
+				ERR("Could not resolve localhost address; destination string is too short");
+				goto error;
+			}
+			strcpy(dst, loopback_addr);
+		} else {
 			/* At this point, the IP or the hostname is bad */
-			ERR("URI parse bad hostname %s for af %d", addr, af);
 			goto error;
 		}
-
-		/* Translate IP to string */
-		(void) inet_ntop(af, record->h_addr_list[0], dst, size);
 	} else {
 		if (size > 0) {
 			strncpy(dst, addr, size);
@@ -132,10 +164,10 @@ static int set_ip_address(const char *addr, int af, char *dst, size_t size)
 	}
 
 	DBG2("IP address resolved to %s", dst);
-
 	return 0;
 
 error:
+	ERR("URI parse bad hostname %s for af %d", addr, af);
 	return -1;
 }