Changeset 7847

Timestamp:

Feb 8, 2009, 12:29:20 AM (12 years ago)

Author:

jhujhiti

Message:

(1.5x libT) backport r7839 (stop using ipv4-mapped listeners)

Location:

branches/1.5x/libtransmission

Files:

• net.c (modified) (3 diffs)
• net.h (modified) (1 diff)
• port-forwarding.c (modified) (1 diff)

branches/1.5x/libtransmission/net.c

@@ -243 +243 @@
 } 
 
-tr_net_af_support
-tr_net_getAFSupport( tr_port port )
-{
-    /* Do we care if an address is in use? Probably not, since it will be
-     * caught later. This will only set up the list of sockets to bind. */
-    static tr_bool alreadyDone       = FALSE;
-    static tr_net_af_support support = { FALSE, FALSE };
-    int s4, s6;
+tr_bool
+tr_net_hasIPv6( tr_port port )
+{
+    static tr_bool alreadyDone = FALSE;
+    static tr_bool result      = FALSE;
+    int s;
     if( alreadyDone )
-        return support;
-    s6 = tr_netBindTCP( &tr_in6addr_any, port, TRUE );
-    if( s6 >= 0 || -s6 != EAFNOSUPPORT ) /* we support ipv6 */
-    {
-        listen( s6, 1 );
-        support.has_inet6 = TRUE;
-    }
-    s4 = tr_netBindTCP( &tr_inaddr_any, port, TRUE );
-    if( s4 >= 0 ) /* we bound *with* the ipv6 socket bound (need both)
-                   * or only have ipv4 */
-    {
-        tr_netClose( s4 );
-        support.needs_inet4 = TRUE;
-    }
-    if( s6 >= 0 )
-        tr_netClose( s6 );
+        return result;
+    s = tr_netBindTCP( &tr_in6addr_any, port, TRUE );
+    if( s >= 0 || -s != EAFNOSUPPORT ) /* we support ipv6 */
+    {
+        result = TRUE;
+        tr_netClose( s );
+    }
     alreadyDone = TRUE;
-    return support;
+    return result;
 }
 
@@ -538 +527 @@
     const int               type = SOCK_STREAM;
     int                     addrlen;
-
-#if defined( SO_REUSEADDR ) || defined( SO_REUSEPORT )
-    int                optval;
+    int                     retval;
+
+#if defined( SO_REUSEADDR ) || defined( SO_REUSEPORT ) || defined( IPV6_V6ONLY )
+    int                optval = 1;
 #endif
 
@@ -550 +540 @@
 
 #ifdef SO_REUSEADDR
-    optval = 1;
     setsockopt( s, SOL_SOCKET, SO_REUSEADDR, (char*)&optval, sizeof( optval ) );
+#endif
+
+#ifdef IPV6_V6ONLY
+    if( retval = setsockopt( s, IPPROTO_IPV6, IPV6_V6ONLY, &optval,
+                sizeof( optval ) ) == -1 ) {
+        /* the kernel may not support this. if not, ignore it */
+        if( errno != ENOPROTOOPT )
+            return -errno;
+    }
 #endif
 

branches/1.5x/libtransmission/net.h

@@ -95 +95 @@
 static TR_INLINE tr_bool tr_isAddress( const tr_address * a ) { return ( a != NULL ) && ( a->type==TR_AF_INET || a->type==TR_AF_INET6 ); }
 
-typedef struct tr_net_af_support
-{
-    tr_bool has_inet6;
-    tr_bool needs_inet4;
-} tr_net_af_support;
-
-tr_net_af_support tr_net_getAFSupport( tr_port );
+tr_bool tr_net_hasIPv6( tr_port );
 
 /***********************************************************************

branches/1.5x/libtransmission/port-forwarding.c

@@ -225 +225 @@
 setupBindSockets( tr_port port )
 {
-    tr_net_af_support support = tr_net_getAFSupport( port );
+    tr_bool hasIPv6 = tr_net_hasIPv6( port );
     tr_socketList * socks = NULL;
-    if( support.has_inet6 )
+    if( hasIPv6 )
         socks = tr_socketListNew( &tr_in6addr_any );
-    if( support.needs_inet4 )
-    {
-        if( socks )
-            tr_socketListAppend( socks, &tr_inaddr_any );
-        else
-            socks = tr_socketListNew( &tr_inaddr_any );
-    }
+
+    if( socks )
+        tr_socketListAppend( socks, &tr_inaddr_any );
+    else
+        socks = tr_socketListNew( &tr_inaddr_any );
     return socks; /* Because the dryer gremlins won't */
 }