]> git.ipfire.org Git - thirdparty/cups.git/blobdiff - cups/http-addrlist.c
Changes to eliminate warnings from new Clang.
[thirdparty/cups.git] / cups / http-addrlist.c
index e62a556f490e51363f2f3613cb2a3bff8cc5489c..9f5cad56d99f8d356a15846a540fcf9b2a46c695 100644 (file)
@@ -1,25 +1,18 @@
 /*
- * "$Id: http-addrlist.c 7910 2008-09-06 00:25:17Z mike $"
+ * "$Id$"
  *
- *   HTTP address list routines for CUPS.
+ * HTTP address list routines for CUPS.
  *
- *   Copyright 2007-2012 by Apple Inc.
- *   Copyright 1997-2007 by Easy Software Products, all rights reserved.
+ * Copyright 2007-2014 by Apple Inc.
+ * Copyright 1997-2007 by Easy Software Products, all rights reserved.
  *
- *   These coded instructions, statements, and computer programs are the
- *   property of Apple Inc. and are protected by Federal copyright
- *   law.  Distribution and use rights are outlined in the file "LICENSE.txt"
- *   which should have been included with this file.  If this file is
- *   file is missing or damaged, see the license at "http://www.cups.org/".
+ * These coded instructions, statements, and computer programs are the
+ * property of Apple Inc. and are protected by Federal copyright
+ * law.  Distribution and use rights are outlined in the file "LICENSE.txt"
+ * which should have been included with this file.  If this file is
+ * file is missing or damaged, see the license at "http://www.cups.org/".
  *
- * Contents:
- *
- *   httpAddrConnect() - Connect to any of the addresses in the list.
- *   httpAddrConnect2() - Connect to any of the addresses in the list with a
- *                       timeout and optional cancel.
- *   httpAddrCopyList() - Copy an address list.
- *   httpAddrFreeList() - Free an address list.
- *   httpAddrGetList() - Get a list of addresses for a hostname.
+ * This file is subject to the Apple OS-Developed Software exception.
  */
 
 /*
@@ -34,7 +27,7 @@
 #  include <poll.h>
 #endif /* HAVE_POLL */
 #ifndef WIN32
-#  include <sys/fcntl.h>
+#  include <fcntl.h>
 #endif /* WIN32 */
 
 
@@ -49,7 +42,7 @@ httpAddrConnect(
     http_addrlist_t *addrlist,         /* I - List of potential addresses */
     int             *sock)             /* O - Socket */
 {
-  DEBUG_printf(("httpAddrConnect(addrlist=%p, sock=%p)", addrlist, sock));
+  DEBUG_printf(("httpAddrConnect(addrlist=%p, sock=%p)", (void *)addrlist, (void *)sock));
 
   return (httpAddrConnect2(addrlist, sock, 30000, NULL));
 }
@@ -59,7 +52,7 @@ httpAddrConnect(
  * 'httpAddrConnect2()' - Connect to any of the addresses in the list with a
  *                        timeout and optional cancel.
  *
- * @since CUPS 1.6/OS X 10.8@
+ * @since CUPS 1.7/OS X 10.9@
  */
 
 http_addrlist_t *                      /* O - Connected address or NULL on failure */
@@ -75,62 +68,70 @@ httpAddrConnect2(
   http_addr_t          peer;           /* Peer address */
   int                  flags,          /* Socket flags */
                        remaining;      /* Remaining timeout */
+  int                  i,              /* Looping var */
+                       nfds,           /* Number of file descriptors */
+                       fds[100],       /* Socket file descriptors */
+                       result;         /* Result from select() or poll() */
+  http_addrlist_t      *addrs[100];    /* Addresses */
 #  ifdef HAVE_POLL
-  struct pollfd                pfd;            /* Polled file descriptor */
+  struct pollfd                pfds[100];      /* Polled file descriptors */
 #  else
+  int                  max_fd = -1;    /* Highest file descriptor */
   fd_set               input_set,      /* select() input set */
                        output_set;     /* select() output set */
   struct timeval       timeout;        /* Timeout */
 #  endif /* HAVE_POLL */
-  int                  nfds;           /* Result from select()/poll() */
 #endif /* O_NONBLOCK */
 #ifdef DEBUG
   char                 temp[256];      /* Temporary address string */
 #endif /* DEBUG */
 
 
-  DEBUG_printf(("httpAddrConnect2(addrlist=%p, sock=%p, msec=%d, cancel=%p)",
-                addrlist, sock, msec, cancel));
+  DEBUG_printf(("httpAddrConnect2(addrlist=%p, sock=%p, msec=%d, cancel=%p)", (void *)addrlist, (void *)sock, msec, (void *)cancel));
 
   if (!sock)
   {
     errno = EINVAL;
-    _cupsSetError(IPP_INTERNAL_ERROR, strerror(errno), 0);
+    _cupsSetError(IPP_STATUS_ERROR_INTERNAL, strerror(errno), 0);
     return (NULL);
   }
 
   if (cancel && *cancel)
     return (NULL);
 
-  if (msec <= 0 || getenv("CUPS_DISABLE_ASYNC_CONNECT"))
+  if (msec <= 0)
     msec = INT_MAX;
 
  /*
   * Loop through each address until we connect or run out of addresses...
   */
 
-  while (addrlist)
+  for (nfds = 0; addrlist && nfds < (int)(sizeof(fds) / sizeof(fds[0])); addrlist = addrlist->next)
   {
     if (cancel && *cancel)
+    {
+      while (nfds > 0)
+      {
+        nfds --;
+       httpAddrClose(NULL, fds[nfds]);
+      }
+
       return (NULL);
+    }
 
    /*
     * Create the socket...
     */
 
-    DEBUG_printf(("2httpAddrConnect2: Trying %s:%d...",
-                 httpAddrString(&(addrlist->addr), temp, sizeof(temp)),
-                 httpAddrPort(&(addrlist->addr))));
+    DEBUG_printf(("2httpAddrConnect2: Trying %s:%d...", httpAddrString(&(addrlist->addr), temp, sizeof(temp)), httpAddrPort(&(addrlist->addr))));
 
-    if ((*sock = (int)socket(_httpAddrFamily(&(addrlist->addr)), SOCK_STREAM,
-                             0)) < 0)
+    if ((fds[nfds] = (int)socket(httpAddrFamily(&(addrlist->addr)), SOCK_STREAM, 0)) < 0)
     {
      /*
       * Don't abort yet, as this could just be an issue with the local
       * system not being configured with IPv4/IPv6/domain socket enabled...
       */
 
-      addrlist = addrlist->next;
       continue;
     }
 
@@ -139,21 +140,16 @@ httpAddrConnect2(
     */
 
     val = 1;
-#ifdef WIN32
-    setsockopt(*sock, SOL_SOCKET, SO_REUSEADDR, (const char *)&val,
-               sizeof(val));
-#else
-    setsockopt(*sock, SOL_SOCKET, SO_REUSEADDR, &val, sizeof(val));
-#endif /* WIN32 */
+    setsockopt(fds[nfds], SOL_SOCKET, SO_REUSEADDR, CUPS_SOCAST &val, sizeof(val));
 
 #ifdef SO_REUSEPORT
     val = 1;
-    setsockopt(*sock, SOL_SOCKET, SO_REUSEPORT, &val, sizeof(val));
+    setsockopt(fds[nfds], SOL_SOCKET, SO_REUSEPORT, CUPS_SOCAST &val, sizeof(val));
 #endif /* SO_REUSEPORT */
 
 #ifdef SO_NOSIGPIPE
     val = 1;
-    setsockopt(*sock, SOL_SOCKET, SO_NOSIGPIPE, &val, sizeof(val));
+    setsockopt(fds[nfds], SOL_SOCKET, SO_NOSIGPIPE, CUPS_SOCAST &val, sizeof(val));
 #endif /* SO_NOSIGPIPE */
 
    /*
@@ -162,19 +158,14 @@ httpAddrConnect2(
     */
 
     val = 1;
-#ifdef WIN32
-    setsockopt(*sock, IPPROTO_TCP, TCP_NODELAY, (const char *)&val,
-               sizeof(val));
-#else
-    setsockopt(*sock, IPPROTO_TCP, TCP_NODELAY, &val, sizeof(val));
-#endif /* WIN32 */
+    setsockopt(fds[nfds], IPPROTO_TCP, TCP_NODELAY, CUPS_SOCAST &val, sizeof(val));
 
 #ifdef FD_CLOEXEC
    /*
     * Close this socket when starting another process...
     */
 
-    fcntl(*sock, F_SETFD, FD_CLOEXEC);
+    fcntl(fds[nfds], F_SETFD, FD_CLOEXEC);
 #endif /* FD_CLOEXEC */
 
 #ifdef O_NONBLOCK
@@ -184,155 +175,161 @@ httpAddrConnect2(
 
     DEBUG_printf(("httpAddrConnect2: Setting non-blocking connect()"));
 
-    flags = fcntl(*sock, F_GETFL, 0);
-    if (msec != INT_MAX)
-    {
-      DEBUG_puts("httpAddrConnect2: Setting non-blocking connect()");
-
-      fcntl(*sock, F_SETFL, flags | O_NONBLOCK);
-    }
+    flags = fcntl(fds[nfds], F_GETFL, 0);
+    fcntl(fds[nfds], F_SETFL, flags | O_NONBLOCK);
 #endif /* O_NONBLOCK */
 
    /*
     * Then connect...
     */
 
-    if (!connect(*sock, &(addrlist->addr.addr),
-                 httpAddrLength(&(addrlist->addr))))
+    if (!connect(fds[nfds], &(addrlist->addr.addr), (socklen_t)httpAddrLength(&(addrlist->addr))))
     {
-      DEBUG_printf(("1httpAddrConnect2: Connected to %s:%d...",
-                   httpAddrString(&(addrlist->addr), temp, sizeof(temp)),
-                   httpAddrPort(&(addrlist->addr))));
+      DEBUG_printf(("1httpAddrConnect2: Connected to %s:%d...", httpAddrString(&(addrlist->addr), temp, sizeof(temp)), httpAddrPort(&(addrlist->addr))));
 
 #ifdef O_NONBLOCK
-      fcntl(*sock, F_SETFL, flags);
+      fcntl(fds[nfds], F_SETFL, flags);
 #endif /* O_NONBLOCK */
 
+      *sock = fds[nfds];
+
+      while (nfds > 0)
+      {
+        nfds --;
+       httpAddrClose(NULL, fds[nfds]);
+      }
+
       return (addrlist);
     }
 
-#ifdef O_NONBLOCK
-#  ifdef WIN32
-    if (WSAGetLastError() == WSAEINPROGRESS ||
-        WSAGetLastError() == WSAEWOULDBLOCK)
-#  else
-    if (errno == EINPROGRESS || errno == EWOULDBLOCK)
-#  endif /* WIN32 */
+#ifdef WIN32
+    if (WSAGetLastError() != WSAEINPROGRESS && WSAGetLastError() != WSAEWOULDBLOCK)
+#else
+    if (errno != EINPROGRESS && errno != EWOULDBLOCK)
+#endif /* WIN32 */
     {
-      DEBUG_puts("1httpAddrConnect2: Finishing async connect()");
+      DEBUG_printf(("1httpAddrConnect2: Unable to connect to %s:%d: %s", httpAddrString(&(addrlist->addr), temp, sizeof(temp)), httpAddrPort(&(addrlist->addr)), strerror(errno)));
+      httpAddrClose(NULL, fds[nfds]);
+      continue;
+    }
+
+    fcntl(fds[nfds], F_SETFL, flags);
 
-      fcntl(*sock, F_SETFL, flags);
+#ifndef HAVE_POLL
+    if (fds[nfds] > max_fd)
+      max_fd = fds[nfds];
+#endif /* !HAVE_POLL */
 
-      for (remaining = msec; remaining > 0; remaining -= 250)
+    addrs[nfds] = addrlist;
+    nfds ++;
+  }
+
+#ifdef O_NONBLOCK
+  DEBUG_puts("1httpAddrConnect2: Finishing async connect()");
+
+  for (remaining = msec; remaining > 0; remaining -= 250)
+  {
+    do
+    {
+      if (cancel && *cancel)
       {
-       do
-        {
-          if (cancel && *cancel)
-          {
-          /*
-           * Close this socket and return...
-           */
+       /*
+       * Close this socket and return...
+       */
 
-            DEBUG_puts("1httpAddrConnect2: Canceled connect()");
+       DEBUG_puts("1httpAddrConnect2: Canceled connect()");
 
-#    ifdef WIN32
-           closesocket(*sock);
-#    else
-           close(*sock);
-#    endif /* WIN32 */
+       while (nfds > 0)
+       {
+         nfds --;
+         httpAddrClose(NULL, fds[nfds]);
+       }
 
-           *sock = -1;
+       *sock = -1;
 
-           return (NULL);
-          }
+       return (NULL);
+      }
 
 #  ifdef HAVE_POLL
-         pfd.fd     = *sock;
-         pfd.events = POLLIN | POLLOUT;
+      for (i = 0; i < nfds; i ++)
+      {
+        pfds[i].fd     = fds[i];
+        pfds[i].events = POLLIN | POLLOUT;
+      }
 
-          nfds = poll(&pfd, 1, remaining > 250 ? 250 : remaining);
+      result = poll(pfds, (nfds_t)nfds, remaining > 250 ? 250 : remaining);
 
-         DEBUG_printf(("1httpAddrConnect2: poll() returned %d (%d)", nfds,
-                       errno));
+      DEBUG_printf(("1httpAddrConnect2: poll() returned %d (%d)", result, errno));
 
 #  else
-         FD_ZERO(&input_set);
-         FD_SET(*sock, &input_set);
-         output_set = input_set;
+      FD_ZERO(&input_set);
+      for (i = 0; i < nfds; i ++)
+        FD_SET(fds[i], &input_set);
+      output_set = input_set;
 
-         timeout.tv_sec  = 0;
-         timeout.tv_usec = (remaining > 250 ? 250 : remaining) * 1000;
+      timeout.tv_sec  = 0;
+      timeout.tv_usec = (remaining > 250 ? 250 : remaining) * 1000;
 
-         nfds = select(*sock + 1, &input_set, &output_set, NULL, &timeout);
+      result = select(max_fd + 1, &input_set, &output_set, NULL, &timeout);
 
-         DEBUG_printf(("1httpAddrConnect2: select() returned %d (%d)", nfds,
-                       errno));
+      DEBUG_printf(("1httpAddrConnect2: select() returned %d (%d)", result, errno));
 #  endif /* HAVE_POLL */
-       }
+    }
 #  ifdef WIN32
-       while (nfds < 0 && (WSAGetLastError() == WSAEINTR ||
-                           WSAGetLastError() == WSAEWOULDBLOCK));
+    while (result < 0 && (WSAGetLastError() == WSAEINTR || WSAGetLastError() == WSAEWOULDBLOCK));
 #  else
-       while (nfds < 0 && (errno == EINTR || errno == EAGAIN));
+    while (result < 0 && (errno == EINTR || errno == EAGAIN));
 #  endif /* WIN32 */
 
-        if (nfds > 0)
+    if (result > 0)
+    {
+      for (i = 0; i < nfds; i ++)
+      {
+#  ifdef HAVE_POLL
+        DEBUG_printf(("pfds[%d].revents=%x\n", i, pfds[i].revents));
+        if (pfds[i].revents)
+#  else
+        if (FD_ISSET(fds[i], &input))
+#  endif /* HAVE_POLL */
         {
-          len = sizeof(peer);
-          if (!getpeername(*sock, (struct sockaddr *)&peer, &len))
-          {
-           DEBUG_printf(("1httpAddrConnect2: Connected to %s:%d...",
-                         httpAddrString(&peer, temp, sizeof(temp)),
-                         httpAddrPort(&peer)));
-
-           return (addrlist);
-         }
+         *sock = fds[i];
+         len   = sizeof(peer);
+         if (!getpeername(fds[i], (struct sockaddr *)&peer, &len))
+         {
+           DEBUG_printf(("1httpAddrConnect2: Connected to %s:%d...", httpAddrString(&peer, temp, sizeof(temp)), httpAddrPort(&peer)));
 
-          break;
-        }
+           addrlist = addrs[i];
+         }
+       }
+       else
+         httpAddrClose(NULL, fds[i]);
       }
+
+      return (addrlist);
     }
+  }
 #endif /* O_NONBLOCK */
 
-    DEBUG_printf(("1httpAddrConnect2: Unable to connect to %s:%d: %s",
-                 httpAddrString(&(addrlist->addr), temp, sizeof(temp)),
-                 httpAddrPort(&(addrlist->addr)), strerror(errno)));
-
-#ifndef WIN32
-    if (errno == EINPROGRESS)
-      errno = ETIMEDOUT;
-#endif /* !WIN32 */
-
-   /*
-    * Close this socket and move to the next address...
-    */
-
-#ifdef WIN32
-    closesocket(*sock);
-#else
-    close(*sock);
-#endif /* WIN32 */
-
-    *sock    = -1;
-    addrlist = addrlist->next;
+  while (nfds > 0)
+  {
+    nfds --;
+    httpAddrClose(NULL, fds[nfds]);
   }
 
-  if (!addrlist)
 #ifdef WIN32
-    _cupsSetError(IPP_SERVICE_UNAVAILABLE, "Connection failed", 0);
+  _cupsSetError(IPP_STATUS_ERROR_SERVICE_UNAVAILABLE, "Connection failed", 0);
 #else
-    _cupsSetError(IPP_SERVICE_UNAVAILABLE, strerror(errno), 0);
+  _cupsSetError(IPP_STATUS_ERROR_SERVICE_UNAVAILABLE, strerror(errno), 0);
 #endif /* WIN32 */
 
-  return (addrlist);
+  return (NULL);
 }
 
 
-
 /*
  * 'httpAddrCopyList()' - Copy an address list.
  *
- * @since CUPS 1.7@
+ * @since CUPS 1.7/OS X 10.9@
  */
 
 http_addrlist_t        *                       /* O - New address list or @code NULL@ on error */
@@ -370,7 +367,8 @@ httpAddrCopyList(
     else
       dst = current;
 
-    src = src->next;
+    prev = current;
+    src  = src->next;
   }
 
   return (dst);
@@ -473,6 +471,7 @@ httpAddrGetList(const char *hostname,       /* I - Hostname, IP address, or NULL for p
 
     if ((first = (http_addrlist_t *)calloc(1, sizeof(http_addrlist_t))) != NULL)
     {
+      addr = first;
       first->addr.un.sun_family = AF_LOCAL;
       strlcpy(first->addr.un.sun_path, hostname, sizeof(first->addr.un.sun_path));
     }
@@ -558,7 +557,7 @@ httpAddrGetList(const char *hostname,       /* I - Hostname, IP address, or NULL for p
          if (!temp)
          {
            httpAddrFreeList(first);
-           _cupsSetError(IPP_INTERNAL_ERROR, strerror(errno), 0);
+           _cupsSetError(IPP_STATUS_ERROR_INTERNAL, strerror(errno), 0);
            return (NULL);
          }
 
@@ -593,7 +592,7 @@ httpAddrGetList(const char *hostname,       /* I - Hostname, IP address, or NULL for p
       if (error == EAI_FAIL)
         cg->need_res_init = 1;
 
-      _cupsSetError(IPP_INTERNAL_ERROR, gai_strerror(error), 0);
+      _cupsSetError(IPP_STATUS_ERROR_INTERNAL, gai_strerror(error), 0);
     }
 
 #else
@@ -656,9 +655,10 @@ httpAddrGetList(const char *hostname,      /* I - Hostname, IP address, or NULL for p
            return (NULL);
 
           first->addr.ipv4.sin_family = AF_INET;
-          first->addr.ipv4.sin_addr.s_addr = htonl(((((((ip[0] << 8) |
-                                                      ip[1]) << 8) |
-                                                    ip[2]) << 8) | ip[3]));
+          first->addr.ipv4.sin_addr.s_addr = htonl((((((((unsigned)ip[0] << 8) |
+                                                      (unsigned)ip[1]) << 8) |
+                                                    (unsigned)ip[2]) << 8) |
+                                                  (unsigned)ip[3]));
           first->addr.ipv4.sin_port = htons(portnum);
        }
       }
@@ -717,7 +717,7 @@ httpAddrGetList(const char *hostname,       /* I - Hostname, IP address, or NULL for p
         if (h_errno == NO_RECOVERY)
           cg->need_res_init = 1;
 
-       _cupsSetError(IPP_INTERNAL_ERROR, hstrerror(h_errno), 0);
+       _cupsSetError(IPP_STATUS_ERROR_INTERNAL, hstrerror(h_errno), 0);
       }
     }
 #endif /* HAVE_GETADDRINFO */
@@ -757,7 +757,7 @@ httpAddrGetList(const char *hostname,       /* I - Hostname, IP address, or NULL for p
     {
       httpAddrFreeList(first);
 
-      _cupsSetError(IPP_INTERNAL_ERROR, _("Unknown service name."), 1);
+      _cupsSetError(IPP_STATUS_ERROR_INTERNAL, _("Unknown service name."), 1);
       return (NULL);
     }
 
@@ -780,7 +780,7 @@ httpAddrGetList(const char *hostname,       /* I - Hostname, IP address, or NULL for p
        temp = (http_addrlist_t *)calloc(1, sizeof(http_addrlist_t));
        if (!temp)
        {
-         _cupsSetError(IPP_INTERNAL_ERROR, strerror(errno), 0);
+         _cupsSetError(IPP_STATUS_ERROR_INTERNAL, strerror(errno), 0);
          httpAddrFreeList(first);
          return (NULL);
        }
@@ -809,7 +809,7 @@ httpAddrGetList(const char *hostname,       /* I - Hostname, IP address, or NULL for p
        temp = (http_addrlist_t *)calloc(1, sizeof(http_addrlist_t));
        if (!temp)
        {
-         _cupsSetError(IPP_INTERNAL_ERROR, strerror(errno), 0);
+         _cupsSetError(IPP_STATUS_ERROR_INTERNAL, strerror(errno), 0);
          httpAddrFreeList(first);
          return (NULL);
        }
@@ -841,7 +841,7 @@ httpAddrGetList(const char *hostname,       /* I - Hostname, IP address, or NULL for p
        temp = (http_addrlist_t *)calloc(1, sizeof(http_addrlist_t));
        if (!temp)
        {
-         _cupsSetError(IPP_INTERNAL_ERROR, strerror(errno), 0);
+         _cupsSetError(IPP_STATUS_ERROR_INTERNAL, strerror(errno), 0);
          httpAddrFreeList(first);
          return (NULL);
        }
@@ -865,7 +865,7 @@ httpAddrGetList(const char *hostname,       /* I - Hostname, IP address, or NULL for p
        temp = (http_addrlist_t *)calloc(1, sizeof(http_addrlist_t));
        if (!temp)
        {
-         _cupsSetError(IPP_INTERNAL_ERROR, strerror(errno), 0);
+         _cupsSetError(IPP_STATUS_ERROR_INTERNAL, strerror(errno), 0);
          httpAddrFreeList(first);
          return (NULL);
        }
@@ -891,5 +891,5 @@ httpAddrGetList(const char *hostname,       /* I - Hostname, IP address, or NULL for p
 
 
 /*
- * End of "$Id: http-addrlist.c 7910 2008-09-06 00:25:17Z mike $".
+ * End of "$Id$".
  */