From 32f60a6e7bec42f733a60eb10afa29a231f691b9 Mon Sep 17 00:00:00 2001
From: Wayne Davison <wayned@samba.org>
Date: Mon, 7 Jul 2003 18:25:01 +0000
Subject: [PATCH] Improved deficiencies in the hostname-vs-address check
 (reintroduced in the last revision) based on the suggestions of Thorild
 Selen.

---
 access.c | 24 ++++++++++++++++++------
 1 file changed, 18 insertions(+), 6 deletions(-)

diff --git a/access.c b/access.c
index 4cbd9a1e..710a1afe 100644
--- a/access.c
+++ b/access.c
@@ -73,16 +73,25 @@ static int match_address(char *addr, char *tok)
 #endif
 	char mask[16];
 	char *a = NULL, *t = NULL;
+	unsigned int len;
 
 	if (!addr || !*addr) return 0;
 
 	p = strchr(tok,'/');
-	if (p) *p = 0;
+	if (p) {
+		*p = '\0';
+		len = p - tok;
+	}
+	else
+		len = strlen(tok);
 
-	/* skip if last char is not a digit (i.e. not an address) */
-	/* (don't check first char - might be 11.22.33.44.an.isp) */
-	if (!*tok) return 0;	/* nothing to check */
-	if (!isdigit(*(unsigned char*)tok+strlen(tok)-1)) return 0;
+	/* Fail quietly if tok is a hostname (not an address) */
+	if (strspn(tok, "./0123456789") != len
+#ifdef INET6
+	 && strspn(tok, "/0123456789:ABCDEFabcdef") != len
+	 && !strchr(tok, '%')
+#endif
+		) return 0;
 
 	memset(&hints, 0, sizeof(hints));
 	hints.ai_family = PF_UNSPEC;
@@ -98,7 +107,10 @@ static int match_address(char *addr, char *tok)
 	if (p)
 		*p++ = '/';
 	if (gai) {
-		rprintf(FERROR,"malformed address %s\n", tok);
+		rprintf(FERROR,
+			"error matching address %s: %s\n",
+			tok,
+			gai_strerror(gai));
 		freeaddrinfo(resa);
 		return 0;
 	}
-- 
2.34.1