Changeset 49197 in vbox for trunk/src/VBox/NetworkServices/NAT

Timestamp:

Oct 20, 2013 10:19:35 PM (11 years ago)

Author:

vboxsync

svn:sync-xref-src-repo-rev:

90080

Message:

Use getaddrinfo(3) on Unix.

File:

• trunk/src/VBox/NetworkServices/NAT/portfwd.c (modified) (3 diffs)

trunk/src/VBox/NetworkServices/NAT/portfwd.c

@@ -5 +5 @@
 #ifndef RT_OS_WINDOWS
 #include <arpa/inet.h>
+#include <netdb.h>
 #include <poll.h>
 #else
@@ -133 +134 @@
            const char *dst_addr_str, uint16_t dst_port)
 {
+    struct addrinfo hints;
+    struct addrinfo *ai;
     int status;
-    int saf;
     void *src_addr, *dst_addr;
 
@@ -143 +145 @@
     fwspec->stype = stype;
 
+    memset(&hints, 0, sizeof(hints));
+    hints.ai_family = (sdom == PF_INET) ? AF_INET : AF_INET6;
+    hints.ai_socktype = stype;
+    hints.ai_flags = AI_NUMERICHOST;
+
+    status = getaddrinfo(src_addr_str, NULL, &hints, &ai);
+    if (status != 0) {
+        DPRINTF(("%s - %s\n", src_addr_str, gai_strerror(status)));
+        return -1;
+    }
+    LWIP_ASSERT1(ai != NULL);
+    LWIP_ASSERT1(ai->ai_addrlen <= sizeof(fwspec->src));
+    memcpy(&fwspec->src, ai->ai_addr, ai->ai_addrlen);
+    freeaddrinfo(ai);
+    ai = NULL;
+
+    status = getaddrinfo(dst_addr_str, NULL, &hints, &ai);
+    if (status != 0) {
+        DPRINTF(("%s - %s\n", dst_addr_str, gai_strerror(status)));
+        return -1;
+    }
+    LWIP_ASSERT1(ai != NULL);
+    LWIP_ASSERT1(ai->ai_addrlen <= sizeof(fwspec->dst));
+    memcpy(&fwspec->dst, ai->ai_addr, ai->ai_addrlen);
+    freeaddrinfo(ai);
+    ai = NULL;
+
     if (sdom == PF_INET) {
-        struct sockaddr_in *src = &fwspec->src.sin;
-        struct sockaddr_in *dst = &fwspec->dst.sin;
-
-        saf = AF_INET;
-
-        src->sin_family = saf;
-#if HAVE_SA_LEN
-        src->sin_len = sizeof(*src);
-#endif
-        src->sin_port = htons(src_port);
-        src_addr = &src->sin_addr;
-
-        dst->sin_family = saf;
-#if HAVE_SA_LEN
-        dst->sin_len = sizeof(*dst);
-#endif
-        dst->sin_port = htons(dst_port);
-        dst_addr = &dst->sin_addr;
+        fwspec->src.sin.sin_port = htons(src_port);
+        fwspec->dst.sin.sin_port = htons(dst_port); 
     }
     else { /* PF_INET6 */
-        struct sockaddr_in6 *src = &fwspec->src.sin6;
-        struct sockaddr_in6 *dst = &fwspec->dst.sin6;
-
-        saf = AF_INET6;
-
-        src->sin6_family = saf;
-#if HAVE_SA_LEN
-        src->sin6_len = sizeof(*src);
-#endif
-        src->sin6_port = htons(src_port);
-        src_addr = &src->sin6_addr;
-
-        dst->sin6_family = saf;
-#if HAVE_SA_LEN
-        dst->sin6_len = sizeof(*dst);
-#endif
-        dst->sin6_port = htons(dst_port);
-        dst_addr = &dst->sin6_addr;
-    }
-
-    status = inet_pton(saf, src_addr_str, src_addr);
-    LWIP_ASSERT1(status >= 0);
-    if (status == 0) {
-        DPRINTF(("bad address: %s\n", src_addr_str));
-        return -1;
-    }
-
-    status = inet_pton(saf, dst_addr_str, dst_addr);
-    LWIP_ASSERT1(status >= 0);
-    if (status == 0) {
-        DPRINTF(("bad address: %s\n", dst_addr_str));
-        return -1;
+        fwspec->src.sin6.sin6_port = htons(src_port);
+        fwspec->dst.sin6.sin6_port = htons(dst_port); 
     }