]> git.sesse.net Git - vlc/blobdiff - src/network/tcp.c
Set non-blocking mode on accepted sockets
[vlc] / src / network / tcp.c
index 828aa381e9b86539c33ac654c4c432b5332f5726..2f851555f7dc95dceda14daed68f1d4a900f17bb 100644 (file)
@@ -26,7 +26,6 @@
 /*****************************************************************************
  * Preamble
  *****************************************************************************/
-#include <stdlib.h>
 #include <vlc/vlc.h>
 
 #include <errno.h>
 #endif
 
 static int SocksNegociate( vlc_object_t *, int fd, int i_socks_version,
-                           char *psz_socks_user, char *psz_socks_passwd );
+                           const char *psz_user, const char *psz_passwd );
 static int SocksHandshakeTCP( vlc_object_t *,
                               int fd, int i_socks_version,
-                              char *psz_socks_user, char *psz_socks_passwd,
+                              const char *psz_user, const char *psz_passwd,
                               const char *psz_host, int i_port );
 extern int net_Socket( vlc_object_t *p_this, int i_family, int i_socktype,
                        int i_protocol );
@@ -75,8 +74,7 @@ int __net_Connect( vlc_object_t *p_this, const char *psz_host, int i_port,
     struct addrinfo hints, *res, *ptr;
     const char      *psz_realhost;
     char            *psz_socks;
-    int             i_realport, i_val, i_handle = -1, i_saved_errno = 0;
-    unsigned        u_errstep = 0;
+    int             i_realport, i_val, i_handle = -1;
 
     if( i_port == 0 )
         i_port = 80; /* historical VLC thing */
@@ -84,8 +82,8 @@ int __net_Connect( vlc_object_t *p_this, const char *psz_host, int i_port,
     memset( &hints, 0, sizeof( hints ) );
     hints.ai_socktype = SOCK_STREAM;
 
-    psz_socks = var_CreateGetString( p_this, "socks" );
-    if( *psz_socks && *psz_socks != ':' )
+    psz_socks = var_CreateGetNonEmptyString( p_this, "socks" );
+    if( psz_socks != NULL )
     {
         char *psz = strchr( psz_socks, ':' );
 
@@ -94,9 +92,35 @@ int __net_Connect( vlc_object_t *p_this, const char *psz_host, int i_port,
 
         psz_realhost = psz_socks;
         i_realport = ( psz != NULL ) ? atoi( psz ) : 1080;
+        hints.ai_flags &= ~AI_NUMERICHOST;
 
-        msg_Dbg( p_this, "net: connecting to %s port %d for %s port %d",
-                 psz_realhost, i_realport, psz_host, i_port );
+        msg_Dbg( p_this, "net: connecting to %s port %d (SOCKS) "
+                 "for %s port %d", psz_realhost, i_realport,
+                 psz_host, i_port );
+
+        /* We only implement TCP with SOCKS */
+        switch( type )
+        {
+            case 0:
+                type = SOCK_STREAM;
+            case SOCK_STREAM:
+                break;
+            default:
+                msg_Err( p_this, "Socket type not supported through SOCKS" );
+                free( psz_socks );
+                return -1;
+        }
+        switch( proto )
+        {
+            case 0:
+                proto = IPPROTO_TCP;
+            case IPPROTO_TCP:
+                break;
+            default:
+                msg_Err( p_this, "Transport not supported through SOCKS" );
+                free( psz_socks );
+                return -1;
+        }
     }
     else
     {
@@ -108,11 +132,12 @@ int __net_Connect( vlc_object_t *p_this, const char *psz_host, int i_port,
     }
 
     i_val = vlc_getaddrinfo( p_this, psz_realhost, i_realport, &hints, &res );
+    free( psz_socks );
+
     if( i_val )
     {
         msg_Err( p_this, "cannot resolve %s port %d : %s", psz_realhost,
                  i_realport, vlc_gai_strerror( i_val ) );
-        free( psz_socks );
         return -1;
     }
 
@@ -122,12 +147,7 @@ int __net_Connect( vlc_object_t *p_this, const char *psz_host, int i_port,
                              proto ?: ptr->ai_protocol );
         if( fd == -1 )
         {
-            if( u_errstep <= 0 )
-            {
-                u_errstep = 1;
-                i_saved_errno = net_errno;
-            }
-            msg_Dbg( p_this, "socket error: %s", strerror( net_errno ) );
+            msg_Dbg( p_this, "socket error: %m" );
             continue;
         }
 
@@ -135,19 +155,14 @@ int __net_Connect( vlc_object_t *p_this, const char *psz_host, int i_port,
         {
             socklen_t i_val_size = sizeof( i_val );
             div_t d;
-            struct timeval tv;
             vlc_value_t timeout;
 
             if( net_errno != EINPROGRESS )
             {
-                if( u_errstep <= 1 )
-                {
-                    u_errstep = 2;
-                    i_saved_errno = net_errno;
-                }
-                msg_Dbg( p_this, "connect error: %s", strerror( net_errno ) );
+                msg_Err( p_this, "connection failed: %m" );
                 goto next_ai;
             }
+            msg_Dbg( p_this, "connection: %m" );
 
             var_Create( p_this, "ipv4-timeout",
                         VLC_VAR_INTEGER | VLC_VAR_DOINHERIT );
@@ -159,10 +174,9 @@ int __net_Connect( vlc_object_t *p_this, const char *psz_host, int i_port,
             }
             d = div( timeout.i_int, 100 );
 
-            msg_Dbg( p_this, "connection in progress" );
             for (;;)
             {
-                fd_set fds;
+                struct pollfd ufd = { .fd = fd, .events = POLLOUT };
                 int i_ret;
 
                 if( p_this->b_die )
@@ -170,41 +184,27 @@ int __net_Connect( vlc_object_t *p_this, const char *psz_host, int i_port,
                     msg_Dbg( p_this, "connection aborted" );
                     net_Close( fd );
                     vlc_freeaddrinfo( res );
-                    free( psz_socks );
                     return -1;
                 }
 
-                /* Initialize file descriptor set */
-                FD_ZERO( &fds );
-                FD_SET( fd, &fds );
-
                 /*
                  * We'll wait 0.1 second if nothing happens
                  * NOTE:
                  * time out will be shortened if we catch a signal (EINTR)
                  */
-                tv.tv_sec = 0;
-                tv.tv_usec = (d.quot > 0) ? 100000 : (1000 * d.rem);
-
-                i_ret = select( fd + 1, NULL, &fds, NULL, &tv );
+                i_ret = poll (&ufd, 1, (d.quot > 0) ? 100 : d.rem);
                 if( i_ret == 1 )
                     break;
 
                 if( ( i_ret == -1 ) && ( net_errno != EINTR ) )
                 {
-                    msg_Warn( p_this, "select error: %s",
-                              strerror( net_errno ) );
+                    msg_Err( p_this, "connection polling error: %m" );
                     goto next_ai;
                 }
 
                 if( d.quot <= 0 )
                 {
-                    msg_Dbg( p_this, "select timed out" );
-                    if( u_errstep <= 2 )
-                    {
-                        u_errstep = 3;
-                        i_saved_errno = ETIMEDOUT;
-                    }
+                    msg_Warn( p_this, "connection timed out" );
                     goto next_ai;
                 }
 
@@ -215,15 +215,14 @@ int __net_Connect( vlc_object_t *p_this, const char *psz_host, int i_port,
             if( getsockopt( fd, SOL_SOCKET, SO_ERROR, (void*)&i_val,
                             &i_val_size ) == -1 || i_val != 0 )
             {
-                u_errstep = 4;
-                i_saved_errno = i_val;
-                msg_Dbg( p_this, "connect error (via getsockopt): %s",
-                         net_strerror( i_val ) );
+                errno = i_val;
+                msg_Err( p_this, "connection failed: %m" );
                 goto next_ai;
             }
 #endif
         }
 
+        msg_Dbg( p_this, "connection succeeded (socket = %d)", fd );
         i_handle = fd; /* success! */
         break;
 
@@ -235,22 +234,18 @@ next_ai: /* failure */
     vlc_freeaddrinfo( res );
 
     if( i_handle == -1 )
-    {
-        msg_Err( p_this, "Connection to %s port %d failed: %s", psz_host,
-                 i_port, net_strerror( i_saved_errno ) );
-        free( psz_socks );
         return -1;
-    }
 
-    if( *psz_socks && *psz_socks != ':' )
+    if( psz_socks != NULL )
     {
-        char *psz_user = var_CreateGetString( p_this, "socks-user" );
-        char *psz_pwd  = var_CreateGetString( p_this, "socks-pwd" );
+        /* NOTE: psz_socks already free'd! */
+        char *psz_user = var_CreateGetNonEmptyString( p_this, "socks-user" );
+        char *psz_pwd  = var_CreateGetNonEmptyString( p_this, "socks-pwd" );
 
         if( SocksHandshakeTCP( p_this, i_handle, 5, psz_user, psz_pwd,
                                psz_host, i_port ) )
         {
-            msg_Err( p_this, "Failed to use the SOCKS server" );
+            msg_Err( p_this, "SOCKS handshake failed" );
             net_Close( i_handle );
             i_handle = -1;
         }
@@ -258,50 +253,73 @@ next_ai: /* failure */
         free( psz_user );
         free( psz_pwd );
     }
-    free( psz_socks );
 
     return i_handle;
 }
 
 
+int net_AcceptSingle (vlc_object_t *obj, int lfd)
+{
+    int fd = accept (lfd, NULL, NULL);
+    if (fd == -1)
+    {
+        if (net_errno != EAGAIN)
+            msg_Err (obj, "accept failed (from socket %d): %m", lfd);
+        return -1;
+    }
+
+    msg_Dbg (obj, "accepted socket %d (from socket %d)", fd, lfd);
+    net_SetupSocket (fd);
+    return 0;
+}
+
+
 /*****************************************************************************
  * __net_Accept:
  *****************************************************************************
  * Accept a connection on a set of listening sockets and return it
  *****************************************************************************/
-int __net_Accept( vlc_object_t *p_this, int pi_fd[], mtime_t i_wait )
+int __net_Accept( vlc_object_t *p_this, int *pi_fd, mtime_t i_wait )
 {
-    vlc_bool_t b_block = (i_wait < 0);
+    int timeout = (i_wait < 0) ? -1 : i_wait / 1000;
+    int evfd;
 
-    while( !p_this->b_die )
+    assert( pi_fd != NULL );
+
+    vlc_object_lock (p_this);
+    evfd = vlc_object_waitpipe (p_this);
+
+    while (vlc_object_alive (p_this))
     {
         unsigned n = 0;
         while (pi_fd[n] != -1)
             n++;
-        struct pollfd ufd[n];
+        struct pollfd ufd[n + 1];
 
         /* Initialize file descriptor set */
-        for (unsigned i = 0; i < n; i++)
+        for (unsigned i = 0; i <= n; i++)
         {
-            ufd[i].fd = pi_fd[i];
+            ufd[i].fd = (i < n) ? pi_fd[i] : evfd;
             ufd[i].events = POLLIN;
             ufd[i].revents = 0;
         }
 
-        switch (poll (ufd, n, b_block ? 500 : i_wait))
+        vlc_object_unlock (p_this);
+        switch (poll (ufd, n, timeout))
         {
             case -1:
                 if (net_errno != EINTR)
-                {
-                    msg_Err (p_this, "poll error: %s",
-                             net_strerror (net_errno));
-                }
-                return -1;
-
+                    msg_Err (p_this, "poll error: %m");
             case 0:
-                if (b_block)
-                    continue;
-                return -1;
+                return -1; /* NOTE: p_this already unlocked */
+        }
+        vlc_object_lock (p_this);
+
+        if (ufd[n].revents)
+        {
+            vlc_object_wait (p_this);
+            errno = EINTR;
+            break;
         }
 
         for (unsigned i = 0; i < n; i++)
@@ -310,14 +328,9 @@ int __net_Accept( vlc_object_t *p_this, int pi_fd[], mtime_t i_wait )
                 continue;
 
             int sfd = ufd[i].fd;
-            int fd = accept (sfd, NULL, NULL);
+            int fd = net_AcceptSingle (p_this, sfd);
             if (fd == -1)
-            {
-                msg_Err (p_this, "accept failed (%s)",
-                         net_strerror (net_errno));
                 continue;
-            }
-            net_SetupSocket (fd);
 
             /*
              * Move listening socket to the end to let the others in the
@@ -325,11 +338,11 @@ int __net_Accept( vlc_object_t *p_this, int pi_fd[], mtime_t i_wait )
              */
             memmove (pi_fd + i, pi_fd + i + 1, n - (i + 1));
             pi_fd[n - 1] = sfd;
-            msg_Dbg (p_this, "accepted socket %d (from socket %d)", fd, sfd);
+            vlc_object_unlock (p_this);
             return fd;
         }
     }
-
+    vlc_object_unlock (p_this);
     return -1;
 }
 
@@ -341,8 +354,8 @@ int __net_Accept( vlc_object_t *p_this, int pi_fd[], mtime_t i_wait )
  *****************************************************************************/
 static int SocksNegociate( vlc_object_t *p_obj,
                            int fd, int i_socks_version,
-                           char *psz_socks_user,
-                           char *psz_socks_passwd )
+                           const char *psz_socks_user,
+                           const char *psz_socks_passwd )
 {
     uint8_t buffer[128+2*256];
     int i_len;
@@ -353,8 +366,7 @@ static int SocksNegociate( vlc_object_t *p_obj,
 
     /* We negociate authentication */
 
-    if( psz_socks_user && psz_socks_passwd &&
-        *psz_socks_user && *psz_socks_passwd )
+    if( ( psz_socks_user == NULL ) && ( psz_socks_passwd == NULL ) )
         b_auth = VLC_TRUE;
 
     buffer[0] = i_socks_version;    /* SOCKS version */
@@ -432,7 +444,7 @@ static int SocksNegociate( vlc_object_t *p_obj,
 static int SocksHandshakeTCP( vlc_object_t *p_obj,
                               int fd,
                               int i_socks_version,
-                              char *psz_socks_user, char *psz_socks_passwd,
+                              const char *psz_user, const char *psz_passwd,
                               const char *psz_host, int i_port )
 {
     uint8_t buffer[128+2*256];
@@ -443,9 +455,9 @@ static int SocksHandshakeTCP( vlc_object_t *p_obj,
         i_socks_version = 5;
     }
 
-    if( i_socks_version == 5 && 
+    if( i_socks_version == 5 &&
         SocksNegociate( p_obj, fd, i_socks_version,
-                        psz_socks_user, psz_socks_passwd ) )
+                        psz_user, psz_passwd ) )
         return VLC_EGENERIC;
 
     if( i_socks_version == 4 )
@@ -453,7 +465,7 @@ static int SocksHandshakeTCP( vlc_object_t *p_obj,
         struct addrinfo hints, *p_res;
 
         /* v4 only support ipv4 */
-       memset (&hints, 0, sizeof (hints));
+        memset (&hints, 0, sizeof (hints));
         hints.ai_family = AF_INET;
         if( vlc_getaddrinfo( p_obj, psz_host, 0, &hints, &p_res ) )
             return VLC_EGENERIC;
@@ -518,7 +530,7 @@ static int SocksHandshakeTCP( vlc_object_t *p_obj,
             i_len = buffer[4] + 2;
         else if( buffer[3] == 0x04 )
             i_len = 16-1+2;
-        else 
+        else
             return VLC_EGENERIC;
 
         if( net_Read( p_obj, fd, NULL, buffer, i_len, VLC_TRUE ) != i_len )