X-Git-Url: https://git.decadent.org.uk/gitweb/?p=nfs-utils.git;a=blobdiff_plain;f=support%2Fexport%2Fhostname.c;h=3957d80069c5510b9faa6e60a4c876d00573ace5;hp=f88e0a485290cb878ab670f4b6917648c3903485;hb=975a8822c32d969fb2cd6108b3ec8e278f6b0e77;hpb=430052cab3c8044ef6d1be7b5a5ded13c45d0c40

diff --git a/support/export/hostname.c b/support/export/hostname.c
index f88e0a4..3957d80 100644
--- a/support/export/hostname.c
+++ b/support/export/hostname.c
@@ -224,46 +224,45 @@ matchhostname (const char *h1, const char *h2)
 struct hostent *
 get_reliable_hostbyaddr(const char *addr, int len, int type)
 {
-	struct hostent *hp;
+	struct hostent *hp = NULL;
 
+	struct hostent *reverse;
+	struct hostent *forward;
 	char **sp;
-	struct hostent *forward = NULL;
-	char *tmpname;
 
-	hp = gethostbyaddr(addr, len , type);
-	if (!hp)
-		return hp;
+	reverse = gethostbyaddr (addr, len, type);
+	if (!reverse)
+		return NULL;
 
 	/* must make sure the hostent is authorative. */
 
-	hp = hostent_dup (hp);
-	tmpname = xstrdup((hp)->h_name);
-	if (tmpname) {
-		forward = gethostbyname(tmpname);
-		free(tmpname);
-	}
+	reverse = hostent_dup (reverse);
+	forward = gethostbyname (reverse->h_name);
+
 	if (forward) {
 		/* now make sure the "addr" is in the list */
 		for (sp = forward->h_addr_list ; *sp ; sp++) {
-			if (memcmp(*sp, addr, forward->h_length)==0)
+			if (memcmp (*sp, addr, forward->h_length) == 0)
 				break;
 		}
-		
-		if (!*sp) {
+
+		if (*sp) {
+			/* it's valid */
+			hp = hostent_dup (forward);
+		}
+		else {
 			/* it was a FAKE */
-			xlog(L_WARNING, "Fake hostname %s for %s - forward lookup doesn't match reverse",
-			     forward->h_name, inet_ntoa(*(struct in_addr*)addr));
-			return NULL;
+			xlog (L_WARNING, "Fake hostname %s for %s - forward lookup doesn't match reverse",
+			      reverse->h_name, inet_ntoa(*(struct in_addr*)addr));
 		}
-		free (hp);
-		hp = hostent_dup (forward);
 	}
 	else {
 		/* never heard of it. misconfigured DNS? */
-		xlog(L_WARNING, "Fake hostname %s for %s - forward lookup doesn't exist",
-		     forward->h_name, inet_ntoa(*(struct in_addr*)addr));
-		return NULL;
+		xlog (L_WARNING, "Fake hostname %s for %s - forward lookup doesn't exist",
+		      reverse->h_name, inet_ntoa(*(struct in_addr*)addr));
 	}
+
+	free (reverse);
 	return hp;
 }