[glibc/siddhesh/gai-cleanup2] gaih_inet: Split simple gethostbyname into its own function

[Siddhesh Poyarekar <siddhesh@sourceware.org>]

https://sourceware.org/git/gitweb.cgi?p=glibc.git;h=3be6e2d43ba425a688b1ab9815d55a450fa857b7

commit 3be6e2d43ba425a688b1ab9815d55a450fa857b7
Author: Siddhesh Poyarekar <siddhesh@sourceware.org>
Date:   Mon Feb 28 14:21:26 2022 +0530

    gaih_inet: Split simple gethostbyname into its own function
    
    Signed-off-by: Siddhesh Poyarekar <siddhesh@sourceware.org>

Diff:
---
 sysdeps/posix/getaddrinfo.c | 138 ++++++++++++++++++++++++--------------------
 1 file changed, 75 insertions(+), 63 deletions(-)

diff --git a/sysdeps/posix/getaddrinfo.c b/sysdeps/posix/getaddrinfo.c
index d3abc59ba9..fbb8139d5a 100644
--- a/sysdeps/posix/getaddrinfo.c
+++ b/sysdeps/posix/getaddrinfo.c
@@ -571,6 +571,67 @@ get_numeric_res (const char *name, const struct addrinfo *req,
   return 0;
 }
 
+/* Call the simple, old functions, which do not support IPv6 scope ids, nor
+   retrieving the canonical name.  */
+
+static int
+simple_gethostbyname (const char *name, const struct addrinfo *req,
+		      struct scratch_buffer *tmpbuf,
+		      struct gaih_lookup_result *res)
+{
+  int rc;
+  struct hostent th;
+  struct hostent *h;
+  struct gaih_addrtuple *at = NULL;
+
+  res->at = NULL;
+
+  while (1)
+    {
+      rc = __gethostbyname2_r (name, AF_INET, &th,
+			       tmpbuf->data, tmpbuf->length,
+			       &h, &h_errno);
+      if (rc != ERANGE || h_errno != NETDB_INTERNAL)
+	break;
+      if (!scratch_buffer_grow (tmpbuf))
+	return -EAI_MEMORY;
+    }
+
+  if (rc == 0)
+    {
+      if (h != NULL)
+	{
+	  /* We found data, convert it.  */
+	  if (!convert_hostent_to_gaih_addrtuple (req, AF_INET, h, &at))
+	    return -EAI_MEMORY;
+
+	  if (at != NULL && !gaih_lookup_result_push_alloc (res, at))
+	    {
+	      free (at);
+	      return -EAI_MEMORY;
+	    }
+
+	  res->at = at;
+	  return 0;
+	}
+
+      if (h_errno == NO_DATA)
+	return -EAI_NODATA;
+
+      return -EAI_NONAME;
+    }
+
+  if (h_errno == NETDB_INTERNAL)
+    return -EAI_SYSTEM;
+
+  if (h_errno == TRY_AGAIN)
+    return -EAI_AGAIN;
+
+  /* We made requests but they turned out no data. The name is known,
+     though.  */
+  return -EAI_NODATA;
+}
+
 static int
 gaih_inet (const char *name, const struct gaih_service *service,
 	   const struct addrinfo *req, struct addrinfo **pai,
@@ -623,6 +684,20 @@ gaih_inet (const char *name, const struct gaih_service *service,
 	  goto process_list;
 	}
 
+      /* If we do not have to look for IPv6 addresses or the canonical
+	 name, use the simple, old functions, which do not support
+	 IPv6 scope ids, nor retrieving the canonical name.  */
+      if (req->ai_family == AF_INET && (req->ai_flags & AI_CANONNAME) == 0)
+	{
+	  if ((result = simple_gethostbyname (name, req, tmpbuf, &res)) != 0)
+	    goto free_and_return;
+	  else if (res.at != NULL)
+	    {
+	      at = res.at;
+	      goto process_list;
+	    }
+	}
+
       struct gaih_addrtuple **pat = &at;
       int no_data = 0;
       int no_inet6_data = 0;
@@ -632,69 +707,6 @@ gaih_inet (const char *name, const struct gaih_service *service,
       int no_more;
       struct resolv_context *res_ctx = NULL;
 
-      /* If we do not have to look for IPv6 addresses or the canonical
-	 name, use the simple, old functions, which do not support
-	 IPv6 scope ids, nor retrieving the canonical name.  */
-      if (req->ai_family == AF_INET
-	  && (req->ai_flags & AI_CANONNAME) == 0)
-	{
-	  int rc;
-	  struct hostent th;
-	  struct hostent *h;
-
-	  while (1)
-	    {
-	      rc = __gethostbyname2_r (name, AF_INET, &th,
-				       tmpbuf->data, tmpbuf->length,
-				       &h, &h_errno);
-	      if (rc != ERANGE || h_errno != NETDB_INTERNAL)
-		break;
-	      if (!scratch_buffer_grow (tmpbuf))
-		{
-		  result = -EAI_MEMORY;
-		  goto free_and_return;
-		}
-	    }
-
-	  if (rc == 0)
-	    {
-	      if (h != NULL)
-		{
-		  /* We found data, convert it.  */
-		  if (!convert_hostent_to_gaih_addrtuple
-		      (req, AF_INET, h, &addrmem))
-		    {
-		      result = -EAI_MEMORY;
-		      goto free_and_return;
-		    }
-		  *pat = addrmem;
-		}
-	      else
-		{
-		  if (h_errno == NO_DATA)
-		    result = -EAI_NODATA;
-		  else
-		    result = -EAI_NONAME;
-		  goto free_and_return;
-		}
-	    }
-	  else
-	    {
-	      if (h_errno == NETDB_INTERNAL)
-		result = -EAI_SYSTEM;
-	      else if (h_errno == TRY_AGAIN)
-		result = -EAI_AGAIN;
-	      else
-		/* We made requests but they turned out no data.
-		   The name is known, though.  */
-		result = -EAI_NODATA;
-
-	      goto free_and_return;
-	    }
-
-	  goto process_list;
-	}
-
 #ifdef USE_NSCD
       if (__nss_not_use_nscd_hosts > 0
 	  && ++__nss_not_use_nscd_hosts > NSS_NSCD_RETRY)