libxtables: remove unnecessary nesting from host_to_ip(6)addr

The error path already terminally returns from the function, so there
is no point in having an explicit else block.

Signed-off-by: Jan Engelhardt <jengelh@inai.de>
Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>

diff --git a/libxtables/xtables.c b/libxtables/xtables.c
index d43f97066ea9b71c21791e7f16de373162bf482c..ae73a06ab30a646c47349de9f772b71291908a7c 100644 (file)
--- a/libxtables/xtables.c
+++ b/libxtables/xtables.c
@@ -1372,21 +1372,18 @@ static struct in_addr *host_to_ipaddr(const char *name, unsigned int *naddr)
        hints.ai_socktype = SOCK_RAW;
 
        *naddr = 0;
-       if ((err = getaddrinfo(name, NULL, &hints, &res)) != 0) {
+       err = getaddrinfo(name, NULL, &hints, &res);
+       if (err != 0)
                return NULL;
-       } else {
-               for (p = res; p != NULL; p = p->ai_next)
-                       ++*naddr;
-               addr = xtables_calloc(*naddr, sizeof(struct in_addr));
-               for (i = 0, p = res; p != NULL; p = p->ai_next)
-                       memcpy(&addr[i++],
-                              &((const struct sockaddr_in *)p->ai_addr)->sin_addr,
-                              sizeof(struct in_addr));
-               freeaddrinfo(res);
-               return addr;
-       }
-
-       return NULL;
+       for (p = res; p != NULL; p = p->ai_next)
+               ++*naddr;
+       addr = xtables_calloc(*naddr, sizeof(struct in_addr));
+       for (i = 0, p = res; p != NULL; p = p->ai_next)
+               memcpy(&addr[i++],
+                      &((const struct sockaddr_in *)p->ai_addr)->sin_addr,
+                      sizeof(struct in_addr));
+       freeaddrinfo(res);
+       return addr;
 }
 
 static struct in_addr *
@@ -1662,23 +1659,20 @@ host_to_ip6addr(const char *name, unsigned int *naddr)
        hints.ai_socktype = SOCK_RAW;
 
        *naddr = 0;
-       if ((err = getaddrinfo(name, NULL, &hints, &res)) != 0) {
+       err = getaddrinfo(name, NULL, &hints, &res);
+       if (err != 0)
                return NULL;
-       } else {
-               /* Find length of address chain */
-               for (p = res; p != NULL; p = p->ai_next)
-                       ++*naddr;
-               /* Copy each element of the address chain */
-               addr = xtables_calloc(*naddr, sizeof(struct in6_addr));
-               for (i = 0, p = res; p != NULL; p = p->ai_next)
-                       memcpy(&addr[i++],
-                              &((const struct sockaddr_in6 *)p->ai_addr)->sin6_addr,
-                              sizeof(struct in6_addr));
-               freeaddrinfo(res);
-               return addr;
-       }
-
-       return NULL;
+       /* Find length of address chain */
+       for (p = res; p != NULL; p = p->ai_next)
+               ++*naddr;
+       /* Copy each element of the address chain */
+       addr = xtables_calloc(*naddr, sizeof(struct in6_addr));
+       for (i = 0, p = res; p != NULL; p = p->ai_next)
+               memcpy(&addr[i++],
+                      &((const struct sockaddr_in6 *)p->ai_addr)->sin6_addr,
+                      sizeof(struct in6_addr));
+       freeaddrinfo(res);
+       return addr;
 }
 
 static struct in6_addr *network_to_ip6addr(const char *name)