Mailing-List: contact dev-help@apr.apache.org; run by ezmlm
Precedence: bulk
Received-SPF: pass (herse.apache.org: local policy)
Message-ID: <4667786D.8040308@haxent.com.br>
Date: Thu, 07 Jun 2007 00:15:57 -0300
From: Davi Arnaut <davi@haxent.com.br>
MIME-Version: 1.0
To: colm@stdlib.net
Cc: "William A. Rowe, Jr." <wrowe@rowe-clan.net>,
	lucian.grijincu@gmail.com, dev@apr.apache.org
Subject: Re: [Vote] Release APR 1.2.9/0.9.14 and apr-iconv 1.2.0
References: <46649707.6010808@rowe-clan.net>
 <4d45da050706051709xcb3c061gfe56c73223a24ffc@mail.gmail.com>
 <466650B1.4060203@rowe-clan.net>
 <4d45da050706060140v73014bdr9784afab278526bd@mail.gmail.com>
 <4d45da050706060439n45645cbexd5aaf6389df6ceb8@mail.gmail.com>
 <4666E358.3020304@haxent.com.br>
 <20070606232144.GD20426@infiltrator.gizzard.com>
 <46674514.1030600@rowe-clan.net>
 <20070606234442.GC21957@infiltrator.gizzard.com>
In-Reply-To: <20070606234442.GC21957@infiltrator.gizzard.com>
Content-Type: multipart/mixed;
 boundary="------------060404030800090501030508"

This is a multi-part message in MIME format.
--------------060404030800090501030508
Content-Type: text/plain; charset=ISO-8859-15
Content-Transfer-Encoding: 7bit

Colm MacCarthaigh wrote:
> On Wed, Jun 06, 2007 at 06:36:52PM -0500, William A. Rowe, Jr. wrote:
>> +1 - that sounds like a great solution.  Do you want me to hold off on
>> the 1.2.10 tag for a day or two to slip this in, now that we know what
>> the case is?
> 
> Nah, it's been like that for ages :-) 
> 

In any case, I like to have a working make test. The patch attached has been
tested on Ubuntu and Fedora.

--
Davi Arnaut

--------------060404030800090501030508
Content-Type: text/plain; x-mac-type="0"; x-mac-creator="0";
 name="apr-ipv6-test-fix.patch"
Content-Transfer-Encoding: 7bit
Content-Disposition: inline;
 filename="apr-ipv6-test-fix.patch"

Index: apr/test/testsockets.c
===================================================================
--- apr.orig/test/testsockets.c
+++ apr/test/testsockets.c
@@ -103,7 +103,7 @@ static void udp6_socket(abts_case *tc, v
 #endif
 }
 
-static void sendto_receivefrom(abts_case *tc, void *data)
+static void sendto_receivefrom_helper(abts_case *tc, const char *addr, int family)
 {
     apr_status_t rv;
     apr_socket_t *sock = NULL;
@@ -115,28 +115,19 @@ static void sendto_receivefrom(abts_case
     apr_sockaddr_t *from;
     apr_sockaddr_t *to;
     apr_size_t len = 30;
-    int family;
-    const char *addr;
 
-#if APR_HAVE_IPV6
-    family = APR_INET6;
-    addr = "::1";
     rv = apr_socket_create(&sock, family, SOCK_DGRAM, 0, p);
-    if (V6_NOT_ENABLED(rv)) {
-#endif
-        family = APR_INET;
-        addr = "127.0.0.1";
-        rv = apr_socket_create(&sock, family, SOCK_DGRAM, 0, p);
-#if APR_HAVE_IPV6
-    } 
-#endif
     ABTS_INT_EQUAL(tc, APR_SUCCESS, rv);
+    if (rv != APR_SUCCESS)
+        return;
     rv = apr_socket_create(&sock2, family, SOCK_DGRAM, 0, p);
     ABTS_INT_EQUAL(tc, APR_SUCCESS, rv);
+    if (rv != APR_SUCCESS)
+        return;
 
-    rv = apr_sockaddr_info_get(&to, addr, APR_UNSPEC, 7772, 0, p);
+    rv = apr_sockaddr_info_get(&to, addr, family, 7772, 0, p);
     ABTS_INT_EQUAL(tc, APR_SUCCESS, rv);
-    rv = apr_sockaddr_info_get(&from, addr, APR_UNSPEC, 7771, 0, p);
+    rv = apr_sockaddr_info_get(&from, addr, family, 7771, 0, p);
     ABTS_INT_EQUAL(tc, APR_SUCCESS, rv);
 
     rv = apr_socket_opt_set(sock, APR_SO_REUSEADDR, 1);
@@ -147,12 +138,12 @@ static void sendto_receivefrom(abts_case
     rv = apr_socket_bind(sock, to);
     APR_ASSERT_SUCCESS(tc, "Could not bind socket", rv);
     if (rv != APR_SUCCESS)
-      return;
+        return;
 
     rv = apr_socket_bind(sock2, from);
     APR_ASSERT_SUCCESS(tc, "Could not bind second socket", rv);
     if (rv != APR_SUCCESS)
-      return;
+        return;
 
     len = STRLEN;
     rv = apr_socket_sendto(sock2, to, 0, sendbuf, &len);
@@ -178,6 +169,14 @@ static void sendto_receivefrom(abts_case
     apr_socket_close(sock2);
 }
 
+static void sendto_receivefrom(abts_case *tc, void *data)
+{
+#if APR_HAVE_IPV6
+    sendto_receivefrom_helper(tc, "::1", APR_INET6);
+#endif
+    sendto_receivefrom_helper(tc, "127.0.0.1", APR_INET);
+}
+
 static void socket_userdata(abts_case *tc, void *data)
 {
     apr_socket_t *sock1, *sock2;

--------------060404030800090501030508--