s3: Change sockaddr util function names for consistency
[samba.git] / source3 / lib / util_sock.c
index 8c5d1b6feca2426c0247c2a9926142e0af2c67ef..605bbf1fb6b4b557e6aff4926cc09f196a87276d 100644 (file)
 
 #include "includes.h"
 
-/* the following 3 client_*() functions are nasty ways of allowing
-   some generic functions to get info that really should be hidden in
-   particular modules */
-static int client_fd = -1;
-/* What to print out on a client disconnect error. */
-static char client_ip_string[INET6_ADDRSTRLEN];
-
-/****************************************************************************
- Return true if a string could be a pure IPv4 address.
-****************************************************************************/
-
-bool is_ipaddress_v4(const char *str)
-{
-       bool pure_address = true;
-       int i;
-
-       for (i=0; pure_address && str[i]; i++) {
-               if (!(isdigit((int)str[i]) || str[i] == '.')) {
-                       pure_address = false;
-               }
-       }
-
-       /* Check that a pure number is not misinterpreted as an IP */
-       pure_address = pure_address && (strchr_m(str, '.') != NULL);
-       return pure_address;
-}
-
-/****************************************************************************
- Interpret an internet address or name into an IP address in 4 byte form.
-****************************************************************************/
-
-uint32 interpret_addr(const char *str)
-{
-       struct hostent *hp;
-       uint32 res;
-
-       if (strcmp(str,"0.0.0.0") == 0)
-               return(0);
-       if (strcmp(str,"255.255.255.255") == 0)
-               return(0xFFFFFFFF);
-
-       /* if it's in the form of an IP address then
-        * get the lib to interpret it */
-       if (is_ipaddress_v4(str)) {
-               res = inet_addr(str);
-       } else {
-               /* otherwise assume it's a network name of some sort and use
-                       sys_gethostbyname */
-               if ((hp = sys_gethostbyname(str)) == 0) {
-                       DEBUG(3,("sys_gethostbyname: Unknown host. %s\n",str));
-                       return 0;
-               }
-
-               if(hp->h_addr == NULL) {
-                       DEBUG(3,("sys_gethostbyname: host address is "
-                               "invalid for host %s\n",str));
-                       return 0;
-               }
-               putip((char *)&res,(char *)hp->h_addr);
-       }
-
-       if (res == (uint32)-1)
-               return(0);
-
-       return(res);
-}
-
-/*******************************************************************
- A convenient addition to interpret_addr().
-******************************************************************/
-
-struct in_addr *interpret_addr2(const char *str)
-{
-       static struct in_addr ret;
-       uint32 a = interpret_addr(str);
-       ret.s_addr = a;
-       return(&ret);
-}
-
 /*******************************************************************
  Map a text hostname or IP address (IPv4 or IPv6) into a
  struct sockaddr_storage.
 ******************************************************************/
 
-bool interpret_string_addr(struct sockaddr_storage *pss, const char *str)
+bool interpret_string_addr(struct sockaddr_storage *pss,
+               const char *str,
+               int flags)
 {
-       int ret;
        struct addrinfo *res = NULL;
-       struct addrinfo hints;
-
-       memset(pss,'\0', sizeof(*pss));
+#if defined(HAVE_IPV6)
+       char addr[INET6_ADDRSTRLEN];
+       unsigned int scope_id = 0;
 
-       memset(&hints, '\0', sizeof(hints));
-       /* By default make sure it supports TCP. */
-       hints.ai_socktype = SOCK_STREAM;
-       hints.ai_flags = AI_ADDRCONFIG;
-
-       ret = getaddrinfo(str, NULL,
-                       &hints,
-                       &res);
-
-       if (ret) {
-               DEBUG(3,("interpret_string_addr: getaddrinfo failed for "
-                       "name %s [%s]\n",
-                       str,
-                       gai_strerror(ret) ));
-               return false;
-       }
+       if (strchr_m(str, ':')) {
+               char *p = strchr_m(str, '%');
 
-       /* Copy the first sockaddr. */
-       memcpy(pss, res->ai_addr, res->ai_addrlen);
-       freeaddrinfo(res);
-       return true;
-}
-
-/*******************************************************************
- Check if an IPv7 is 127.0.0.1
-******************************************************************/
-
-bool is_loopback_ip_v4(struct in_addr ip)
-{
-       struct in_addr a;
-       a.s_addr = htonl(INADDR_LOOPBACK);
-       return(ip.s_addr == a.s_addr);
-}
-
-/*******************************************************************
- Check if a struct sockaddr_storage is the loopback address.
-******************************************************************/
+               /*
+                * Cope with link-local.
+                * This is IP:v6:addr%ifname.
+                */
 
-bool is_loopback_addr(const struct sockaddr_storage *pss)
-{
-#if defined(AF_INET6)
-       if (pss->ss_family == AF_INET) {
-               struct in6_addr *pin6 = &((struct sockaddr_in6 *)pss)->sin6_addr;
-               return IN6_IS_ADDR_LOOPBACK(pin6);
+               if (p && (p > str) && ((scope_id = if_nametoindex(p+1)) != 0)) {
+                       strlcpy(addr, str,
+                               MIN(PTR_DIFF(p,str)+1,
+                                       sizeof(addr)));
+                       str = addr;
+               }
        }
 #endif
-       if (pss->ss_family == AF_INET) {
-               struct in_addr *pin = &((struct sockaddr_in *)pss)->sin_addr;
-               return is_loopback_ip_v4(*pin);
-       }
-       return false;
-}
 
-/*******************************************************************
- Check if an IPv4 is 0.0.0.0.
-******************************************************************/
+       zero_sockaddr(pss);
 
-bool is_zero_ip_v4(struct in_addr ip)
-{
-       uint32 a;
-       putip((char *)&a,(char *)&ip);
-       return(a == 0);
-}
-
-/*******************************************************************
- Check if a struct sockaddr_storage has an unspecified address.
-******************************************************************/
-
-bool is_zero_addr(const struct sockaddr_storage *pss)
-{
-#if defined(AF_INET6)
-       if (pss->ss_family == AF_INET) {
-               struct in6_addr *pin6 = &((struct sockaddr_in6 *)pss)->sin6_addr;
-               return IN6_IS_ADDR_UNSPECIFIED(pin6);
-       }
-#endif
-       if (pss->ss_family == AF_INET) {
-               struct in_addr *pin = &((struct sockaddr_in *)pss)->sin_addr;
-               return is_zero_ip_v4(*pin);
+       if (!interpret_string_addr_internal(&res, str, flags|AI_ADDRCONFIG)) {
+               return false;
        }
-       return false;
-}
-
-/*******************************************************************
- Set an IP to 0.0.0.0.
-******************************************************************/
-
-void zero_ip_v4(struct in_addr *ip)
-{
-        static bool init;
-        static struct in_addr ipzero;
-
-        if (!init) {
-                ipzero = *interpret_addr2("0.0.0.0");
-                init = true;
-        }
-
-        *ip = ipzero;
-}
-
-/*******************************************************************
- Are two IPs on the same subnet - IPv4 version ?
-********************************************************************/
-
-bool same_net_v4(struct in_addr ip1,struct in_addr ip2,struct in_addr mask)
-{
-       uint32 net1,net2,nmask;
-
-       nmask = ntohl(mask.s_addr);
-       net1  = ntohl(ip1.s_addr);
-       net2  = ntohl(ip2.s_addr);
-
-       return((net1 & nmask) == (net2 & nmask));
-}
-
-/*******************************************************************
- Convert an IPv4 struct in_addr to a struct sockaddr_storage.
-********************************************************************/
-
-void in_addr_to_sockaddr_storage(struct sockaddr_storage *ss,
-               struct in_addr ip)
-{
-       struct sockaddr_in *sa = (struct sockaddr_in *)ss;
-       memset(ss, '\0', sizeof(*ss));
-       ss->ss_family = AF_INET;
-       sa->sin_addr = ip;
-}
-
-#ifdef AF_INET6
-/*******************************************************************
- Convert an IPv6 struct in_addr to a struct sockaddr_storage.
-********************************************************************/
-
-void in6_addr_to_sockaddr_storage(struct sockaddr_storage *ss,
-               struct in6_addr ip)
-{
-       struct sockaddr_in6 *sa = (struct sockaddr_in6 *)ss;
-       memset(ss, '\0', sizeof(*ss));
-       ss->ss_family = AF_INET6;
-       sa->sin6_addr = ip;
-}
-#endif
-
-/*******************************************************************
- Are two IPs on the same subnet?
-********************************************************************/
-
-bool same_net(const struct sockaddr_storage *ip1,
-               const struct sockaddr_storage *ip2,
-               const struct sockaddr_storage *mask)
-{
-       if (ip1->ss_family != ip2->ss_family) {
-               /* Never on the same net. */
+       if (!res) {
                return false;
        }
+       /* Copy the first sockaddr. */
+       memcpy(pss, res->ai_addr, res->ai_addrlen);
 
-#ifdef AF_INET6
-       if (ip1->ss_family == AF_INET6) {
-               struct sockaddr_in6 ip1_6 = *(struct sockaddr_in6 *)ip1;
-               struct sockaddr_in6 ip2_6 = *(struct sockaddr_in6 *)ip2;
-               struct sockaddr_in6 mask_6 = *(struct sockaddr_in6 *)mask;
-               char *p1 = (char *)&ip1_6.sin6_addr;
-               char *p2 = (char *)&ip2_6.sin6_addr;
-               char *m = (char *)&mask_6.sin6_addr;
-               int i;
-
-               for (i = 0; i < sizeof(struct in6_addr); i++) {
-                       *p1++ &= *m;
-                       *p2++ &= *m;
-                       m++;
+#if defined(HAVE_IPV6)
+       if (pss->ss_family == AF_INET6 && scope_id) {
+               struct sockaddr_in6 *ps6 = (struct sockaddr_in6 *)pss;
+               if (IN6_IS_ADDR_LINKLOCAL(&ps6->sin6_addr) &&
+                               ps6->sin6_scope_id == 0) {
+                       ps6->sin6_scope_id = scope_id;
                }
-               return (memcmp(&ip1_6.sin6_addr,
-                               &ip2_6.sin6_addr,
-                               sizeof(struct in6_addr)) == 0);
        }
 #endif
-       if (ip1->ss_family == AF_INET) {
-               return same_net_v4(((const struct sockaddr_in *)ip1)->sin_addr,
-                               ((const struct sockaddr_in *)ip2)->sin_addr,
-                               ((const struct sockaddr_in *)mask)->sin_addr);
-       }
-       return false;
+
+       freeaddrinfo(res);
+       return true;
 }
 
 /*******************************************************************
Are two sockaddr_storage's the same family and address ? Ignore port etc.
-********************************************************************/
Set an address to INADDR_ANY.
+******************************************************************/
 
-bool addr_equal(const struct sockaddr_storage *ip1,
-               const struct sockaddr_storage *ip2)
+void zero_sockaddr(struct sockaddr_storage *pss)
 {
-       if (ip1->ss_family != ip2->ss_family) {
-               /* Never the same. */
-               return false;
-       }
-
-#ifdef AF_INET6
-       if (ip1->ss_family == AF_INET6) {
-               return (memcmp(&((const struct sockaddr_in6 *)ip1)->sin6_addr,
-                               &((const struct sockaddr_in6 *)ip2)->sin6_addr,
-                               sizeof(struct in6_addr)) == 0);
-       }
-#endif
-       if (ip1->ss_family == AF_INET) {
-               return (memcmp(&((const struct sockaddr_in *)ip1)->sin_addr,
-                               &((const struct sockaddr_in *)ip2)->sin_addr,
-                               sizeof(struct in_addr)) == 0);
-       }
-       return false;
+       memset(pss, '\0', sizeof(*pss));
+       /* Ensure we're at least a valid sockaddr-storage. */
+       pss->ss_family = AF_INET;
 }
 
-
 /****************************************************************************
- Is an IP address the INADDR_ANY or in6addr_any value ?
+ Get a port number in host byte order from a sockaddr_storage.
 ****************************************************************************/
 
-bool is_address_any(const struct sockaddr_storage *psa)
+uint16_t get_sockaddr_port(const struct sockaddr_storage *pss)
 {
-#ifdef AF_INET6
-       if (psa->ss_family == AF_INET6) {
-               struct sockaddr_in6 *si6 = (struct sockaddr_in6 *)psa;
-               if (memcmp(&in6addr_any,
-                               &si6->sin6_addr,
-                               sizeof(in6addr_any)) == 0) {
-                       return true;
-               }
-               return false;
-       }
+       uint16_t port = 0;
+
+       if (pss->ss_family != AF_INET) {
+#if defined(HAVE_IPV6)
+               /* IPv6 */
+               const struct sockaddr_in6 *sa6 =
+                       (const struct sockaddr_in6 *)pss;
+               port = ntohs(sa6->sin6_port);
 #endif
-       if (psa->ss_family == AF_INET) {
-               struct sockaddr_in *si = (struct sockaddr_in *)psa;
-               if (si->sin_addr.s_addr == INADDR_ANY) {
-                       return true;
-               }
-               return false;
+       } else {
+               const struct sockaddr_in *sa =
+                       (const struct sockaddr_in *)pss;
+               port = ntohs(sa->sin_port);
        }
-       return false;
+       return port;
 }
 
 /****************************************************************************
  Print out an IPv4 or IPv6 address from a struct sockaddr_storage.
 ****************************************************************************/
 
-char *print_sockaddr(char *dest,
+static char *print_sockaddr_len(char *dest,
                        size_t destlen,
-                       const struct sockaddr_storage *psa,
+                       const struct sockaddr *psa,
                        socklen_t psalen)
 {
        if (destlen > 0) {
                dest[0] = '\0';
        }
-       (void)getnameinfo((const struct sockaddr *)psa,
+       (void)sys_getnameinfo(psa,
                        psalen,
                        dest, destlen,
                        NULL, 0,
@@ -373,33 +132,68 @@ char *print_sockaddr(char *dest,
 }
 
 /****************************************************************************
- Set the global client_fd variable.
+ Print out an IPv4 or IPv6 address from a struct sockaddr_storage.
+****************************************************************************/
+
+char *print_sockaddr(char *dest,
+                       size_t destlen,
+                       const struct sockaddr_storage *psa)
+{
+       return print_sockaddr_len(dest, destlen, (struct sockaddr *)psa,
+                       sizeof(struct sockaddr_storage));
+}
+
+/****************************************************************************
+ Print out a canonical IPv4 or IPv6 address from a struct sockaddr_storage.
 ****************************************************************************/
 
-void client_setfd(int fd)
+char *print_canonical_sockaddr(TALLOC_CTX *ctx,
+                       const struct sockaddr_storage *pss)
 {
-       client_fd = fd;
-       safe_strcpy(client_ip_string,
-                       get_peer_addr(client_fd),
-                       sizeof(client_ip_string)-1);
+       char addr[INET6_ADDRSTRLEN];
+       char *dest = NULL;
+       int ret;
+
+       /* Linux getnameinfo() man pages says port is unitialized if
+          service name is NULL. */
+
+       ret = sys_getnameinfo((const struct sockaddr *)pss,
+                       sizeof(struct sockaddr_storage),
+                       addr, sizeof(addr),
+                       NULL, 0,
+                       NI_NUMERICHOST);
+       if (ret != 0) {
+               return NULL;
+       }
+
+       if (pss->ss_family != AF_INET) {
+#if defined(HAVE_IPV6)
+               dest = talloc_asprintf(ctx, "[%s]", addr);
+#else
+               return NULL;
+#endif
+       } else {
+               dest = talloc_asprintf(ctx, "%s", addr);
+       }
+       
+       return dest;
 }
 
 /****************************************************************************
- Return a static string of an IP address (IPv4 or IPv6).
+ Return the string of an IP address (IPv4 or IPv6).
 ****************************************************************************/
 
-static const char *get_socket_addr(int fd)
+static const char *get_socket_addr(int fd, char *addr_buf, size_t addr_len)
 {
        struct sockaddr_storage sa;
        socklen_t length = sizeof(sa);
-       static char addr_buf[INET6_ADDRSTRLEN];
 
        /* Ok, returning a hard coded IPv4 address
         * is bogus, but it's just as bogus as a
         * zero IPv6 address. No good choice here.
         */
 
-       safe_strcpy(addr_buf, "0.0.0.0", sizeof(addr_buf)-1);
+       strlcpy(addr_buf, "0.0.0.0", addr_len);
 
        if (fd == -1) {
                return addr_buf;
@@ -411,9 +205,11 @@ static const char *get_socket_addr(int fd)
                return addr_buf;
        }
 
-       return print_sockaddr(addr_buf, sizeof(addr_buf), &sa, length);
+       return print_sockaddr_len(addr_buf, addr_len, (struct sockaddr *)&sa, length);
 }
 
+#if 0
+/* Not currently used. JRA. */
 /****************************************************************************
  Return the port number we've bound to on a socket.
 ****************************************************************************/
@@ -433,7 +229,7 @@ static int get_socket_port(int fd)
                return -1;
        }
 
-#ifdef AF_INET6
+#if defined(HAVE_IPV6)
        if (sa.ss_family == AF_INET6) {
                return ntohs(((struct sockaddr_in6 *)&sa)->sin6_port);
        }
@@ -443,28 +239,50 @@ static int get_socket_port(int fd)
        }
        return -1;
 }
+#endif
 
-const char *client_name(void)
+const char *client_name(int fd)
 {
-       return get_peer_name(client_fd,false);
+       return get_peer_name(fd,false);
 }
 
-const char *client_addr(void)
+const char *client_addr(int fd, char *addr, size_t addrlen)
 {
-       return get_peer_addr(client_fd);
+       return get_peer_addr(fd,addr,addrlen);
 }
 
-const char *client_socket_addr(void)
+const char *client_socket_addr(int fd, char *addr, size_t addr_len)
 {
-       return get_socket_addr(client_fd);
+       return get_socket_addr(fd, addr, addr_len);
 }
 
-int client_socket_port(void)
+#if 0
+/* Not currently used. JRA. */
+int client_socket_port(int fd)
+{
+       return get_socket_port(fd);
+}
+#endif
+
+/****************************************************************************
+ Accessor functions to make thread-safe code easier later...
+****************************************************************************/
+
+void set_smb_read_error(enum smb_read_errors *pre,
+                       enum smb_read_errors newerr)
 {
-       return get_socket_port(client_fd);
+       if (pre) {
+               *pre = newerr;
+       }
 }
 
-int smb_read_error = 0;
+void cond_set_smb_read_error(enum smb_read_errors *pre,
+                       enum smb_read_errors newerr)
+{
+       if (pre && *pre == SMB_READ_OK) {
+               *pre = newerr;
+       }
+}
 
 /****************************************************************************
  Determine if a file descriptor is in fact a socket.
@@ -569,9 +387,10 @@ static void print_socket_options(int s)
 
 void set_socket_options(int fd, const char *options)
 {
-       fstring tok;
+       TALLOC_CTX *ctx = talloc_stackframe();
+       char *tok;
 
-       while (next_token(&options,tok," \t,", sizeof(tok))) {
+       while (next_token_talloc(ctx, &options, &tok," \t,")) {
                int ret=0,i;
                int value = 1;
                char *p;
@@ -615,11 +434,15 @@ void set_socket_options(int fd, const char *options)
                }
 
                if (ret != 0) {
-                       DEBUG(0,("Failed to set socket option %s (Error %s)\n",
+                       /* be aware that some systems like Solaris return
+                        * EINVAL to a setsockopt() call when the client
+                        * sent a RST previously - no need to worry */
+                       DEBUG(2,("Failed to set socket option %s (Error %s)\n",
                                tok, strerror(errno) ));
                }
        }
 
+       TALLOC_FREE(ctx);
        print_socket_options(fd);
 }
 
@@ -672,23 +495,21 @@ ssize_t read_udp_v4_socket(int fd,
  time_out = timeout in milliseconds
 ****************************************************************************/
 
-ssize_t read_socket_with_timeout(int fd,
-                               char *buf,
-                               size_t mincnt,
-                               size_t maxcnt,
-                               unsigned int time_out)
+NTSTATUS read_socket_with_timeout(int fd, char *buf,
+                                 size_t mincnt, size_t maxcnt,
+                                 unsigned int time_out,
+                                 size_t *size_ret)
 {
        fd_set fds;
        int selrtn;
        ssize_t readret;
        size_t nread = 0;
        struct timeval timeout;
+       char addr[INET6_ADDRSTRLEN];
 
        /* just checking .... */
        if (maxcnt <= 0)
-               return(0);
-
-       smb_read_error = 0;
+               return NT_STATUS_OK;
 
        /* Blocking read */
        if (time_out == 0) {
@@ -702,29 +523,27 @@ ssize_t read_socket_with_timeout(int fd,
                        if (readret == 0) {
                                DEBUG(5,("read_socket_with_timeout: "
                                        "blocking read. EOF from client.\n"));
-                               smb_read_error = READ_EOF;
-                               return -1;
+                               return NT_STATUS_END_OF_FILE;
                        }
 
                        if (readret == -1) {
-                               if (fd == client_fd) {
+                               if (fd == get_client_fd()) {
                                        /* Try and give an error message
                                         * saying what client failed. */
                                        DEBUG(0,("read_socket_with_timeout: "
                                                "client %s read error = %s.\n",
-                                               client_ip_string,
+                                               get_peer_addr(fd,addr,sizeof(addr)),
                                                strerror(errno) ));
                                } else {
                                        DEBUG(0,("read_socket_with_timeout: "
                                                "read error = %s.\n",
                                                strerror(errno) ));
                                }
-                               smb_read_error = READ_ERROR;
-                               return -1;
+                               return map_nt_error_from_unix(errno);
                        }
                        nread += readret;
                }
-               return((ssize_t)nread);
+               goto done;
        }
 
        /* Most difficult - timeout read */
@@ -746,27 +565,26 @@ ssize_t read_socket_with_timeout(int fd,
                /* Check if error */
                if (selrtn == -1) {
                        /* something is wrong. Maybe the socket is dead? */
-                       if (fd == client_fd) {
+                       if (fd == get_client_fd()) {
                                /* Try and give an error message saying
                                 * what client failed. */
                                DEBUG(0,("read_socket_with_timeout: timeout "
                                "read for client %s. select error = %s.\n",
-                               client_ip_string, strerror(errno) ));
+                               get_peer_addr(fd,addr,sizeof(addr)),
+                               strerror(errno) ));
                        } else {
                                DEBUG(0,("read_socket_with_timeout: timeout "
                                "read. select error = %s.\n",
                                strerror(errno) ));
                        }
-                       smb_read_error = READ_ERROR;
-                       return -1;
+                       return map_nt_error_from_unix(errno);
                }
 
                /* Did we timeout ? */
                if (selrtn == 0) {
                        DEBUG(10,("read_socket_with_timeout: timeout read. "
                                "select timed out.\n"));
-                       smb_read_error = READ_TIMEOUT;
-                       return -1;
+                       return NT_STATUS_IO_TIMEOUT;
                }
 
                readret = sys_read(fd, buf+nread, maxcnt-nread);
@@ -775,77 +593,44 @@ ssize_t read_socket_with_timeout(int fd,
                        /* we got EOF on the file descriptor */
                        DEBUG(5,("read_socket_with_timeout: timeout read. "
                                "EOF from client.\n"));
-                       smb_read_error = READ_EOF;
-                       return -1;
+                       return NT_STATUS_END_OF_FILE;
                }
 
                if (readret == -1) {
                        /* the descriptor is probably dead */
-                       if (fd == client_fd) {
+                       if (fd == get_client_fd()) {
                                /* Try and give an error message
                                 * saying what client failed. */
                                DEBUG(0,("read_socket_with_timeout: timeout "
                                        "read to client %s. read error = %s.\n",
-                                       client_ip_string, strerror(errno) ));
+                                       get_peer_addr(fd,addr,sizeof(addr)),
+                                       strerror(errno) ));
                        } else {
                                DEBUG(0,("read_socket_with_timeout: timeout "
                                        "read. read error = %s.\n",
                                        strerror(errno) ));
                        }
-                       smb_read_error = READ_ERROR;
-                       return -1;
+                       return map_nt_error_from_unix(errno);
                }
 
                nread += readret;
        }
 
+ done:
        /* Return the number we got */
-       return (ssize_t)nread;
+       if (size_ret) {
+               *size_ret = nread;
+       }
+       return NT_STATUS_OK;
 }
 
 /****************************************************************************
  Read data from the client, reading exactly N bytes.
 ****************************************************************************/
 
-ssize_t read_data(int fd,char *buffer,size_t N)
+NTSTATUS read_data(int fd, char *buffer, size_t N)
 {
-       ssize_t ret;
-       size_t total=0;
-
-       smb_read_error = 0;
-
-       while (total < N) {
-               ret = sys_read(fd,buffer + total,N - total);
-
-               if (ret == 0) {
-                       DEBUG(10,("read_data: read of %d returned 0. "
-                               "Error = %s\n",
-                               (int)(N - total), strerror(errno) ));
-                       smb_read_error = READ_EOF;
-                       return 0;
-               }
-
-               if (ret == -1) {
-                       if (fd == client_fd) {
-                               /* Try and give an error message saying
-                                * what client failed. */
-                               DEBUG(0,("read_data: read failure for %d "
-                                       "bytes to client %s. Error = %s\n",
-                                       (int)(N - total),
-                                       client_ip_string,
-                                       strerror(errno) ));
-                       } else {
-                               DEBUG(0,("read_data: read failure for %d. "
-                                       "Error = %s\n",
-                                       (int)(N - total),
-                                       strerror(errno) ));
-                       }
-                       smb_read_error = READ_ERROR;
-                       return -1;
-               }
-               total += ret;
-       }
-       return (ssize_t)total;
+       return read_socket_with_timeout(fd, buffer, N, N, 0, NULL);
 }
 
 /****************************************************************************
@@ -856,17 +641,19 @@ ssize_t write_data(int fd, const char *buffer, size_t N)
 {
        size_t total=0;
        ssize_t ret;
+       char addr[INET6_ADDRSTRLEN];
 
        while (total < N) {
                ret = sys_write(fd,buffer + total,N - total);
 
                if (ret == -1) {
-                       if (fd == client_fd) {
+                       if (fd == get_client_fd()) {
                                /* Try and give an error message saying
                                 * what client failed. */
                                DEBUG(0,("write_data: write failure in "
                                        "writing to client %s. Error %s\n",
-                                       client_ip_string, strerror(errno) ));
+                                       get_peer_addr(fd,addr,sizeof(addr)),
+                                       strerror(errno) ));
                        } else {
                                DEBUG(0,("write_data: write failure. "
                                        "Error = %s\n", strerror(errno) ));
@@ -905,36 +692,29 @@ bool send_keepalive(int client)
  Timeout is in milliseconds.
 ****************************************************************************/
 
-static ssize_t read_smb_length_return_keepalive(int fd,
-                                               char *inbuf,
-                                               unsigned int timeout)
+NTSTATUS read_smb_length_return_keepalive(int fd, char *inbuf,
+                                         unsigned int timeout,
+                                         size_t *len)
 {
-       ssize_t len=0;
        int msg_type;
-       bool ok = false;
+       NTSTATUS status;
 
-       while (!ok) {
-               if (timeout > 0) {
-                       ok = (read_socket_with_timeout(fd,inbuf,4,4,timeout)
-                                       == 4);
-               } else {
-                       ok = (read_data(fd,inbuf,4) == 4);
-               }
-               if (!ok) {
-                       return -1;
-               }
+       status = read_socket_with_timeout(fd, inbuf, 4, 4, timeout, NULL);
 
-               len = smb_len(inbuf);
-               msg_type = CVAL(inbuf,0);
+       if (!NT_STATUS_IS_OK(status)) {
+               return status;
+       }
 
-               if (msg_type == SMBkeepalive) {
-                       DEBUG(5,("Got keepalive packet\n"));
-               }
+       *len = smb_len(inbuf);
+       msg_type = CVAL(inbuf,0);
+
+       if (msg_type == SMBkeepalive) {
+               DEBUG(5,("Got keepalive packet\n"));
        }
 
-       DEBUG(10,("got smb length of %lu\n",(unsigned long)len));
+       DEBUG(10,("got smb length of %lu\n",(unsigned long)(*len)));
 
-       return(len);
+       return NT_STATUS_OK;
 }
 
 /****************************************************************************
@@ -944,81 +724,55 @@ static ssize_t read_smb_length_return_keepalive(int fd,
  Timeout is in milliseconds.
 ****************************************************************************/
 
-ssize_t read_smb_length(int fd, char *inbuf, unsigned int timeout)
+NTSTATUS read_smb_length(int fd, char *inbuf, unsigned int timeout,
+                        size_t *len)
 {
-       ssize_t len;
+       uint8_t msgtype = SMBkeepalive;
 
-       for(;;) {
-               len = read_smb_length_return_keepalive(fd, inbuf, timeout);
+       while (msgtype == SMBkeepalive) {
+               NTSTATUS status;
 
-               if(len < 0)
-                       return len;
+               status = read_smb_length_return_keepalive(fd, inbuf, timeout,
+                                                         len);
+               if (!NT_STATUS_IS_OK(status)) {
+                       return status;
+               }
 
-               /* Ignore session keepalives. */
-               if(CVAL(inbuf,0) != SMBkeepalive)
-                       break;
+               msgtype = CVAL(inbuf, 0);
        }
 
        DEBUG(10,("read_smb_length: got smb length of %lu\n",
                  (unsigned long)len));
 
-       return len;
+       return NT_STATUS_OK;
 }
 
 /****************************************************************************
- Read an smb from a fd. Note that the buffer *MUST* be of size
- BUFFER_SIZE+SAFETY_MARGIN.
+ Read an smb from a fd.
  The timeout is in milliseconds.
  This function will return on receipt of a session keepalive packet.
  maxlen is the max number of bytes to return, not including the 4 byte
- length. If zero it means BUFFER_SIZE+SAFETY_MARGIN limit.
+ length. If zero it means buflen limit.
  Doesn't check the MAC on signed packets.
 ****************************************************************************/
 
-ssize_t receive_smb_raw(int fd,
-                       char *buffer,
-                       unsigned int timeout,
-                       size_t maxlen)
+NTSTATUS receive_smb_raw(int fd, char *buffer, size_t buflen, unsigned int timeout,
+                        size_t maxlen, size_t *p_len)
 {
-       ssize_t len,ret;
-
-       smb_read_error = 0;
+       size_t len;
+       NTSTATUS status;
 
-       len = read_smb_length_return_keepalive(fd,buffer,timeout);
-       if (len < 0) {
-               DEBUG(10,("receive_smb_raw: length < 0!\n"));
+       status = read_smb_length_return_keepalive(fd,buffer,timeout,&len);
 
-               /*
-                * Correct fix. smb_read_error may have already been
-                * set. Only set it here if not already set. Global
-                * variables still suck :-). JRA.
-                */
-
-               if (smb_read_error == 0)
-                       smb_read_error = READ_ERROR;
-               return -1;
+       if (!NT_STATUS_IS_OK(status)) {
+               DEBUG(10, ("receive_smb_raw: %s!\n", nt_errstr(status)));
+               return status;
        }
 
-       /*
-        * A WRITEX with CAP_LARGE_WRITEX can be 64k worth of data plus 65 bytes
-        * of header. Don't print the error if this fits.... JRA.
-        */
-
-       if (len > (BUFFER_SIZE + LARGE_WRITEX_HDR_SIZE)) {
+       if (len > buflen) {
                DEBUG(0,("Invalid packet length! (%lu bytes).\n",
                                        (unsigned long)len));
-               if (len > BUFFER_SIZE + (SAFETY_MARGIN/2)) {
-
-                       /*
-                        * Correct fix. smb_read_error may have already been
-                        * set. Only set it here if not already set. Global
-                        * variables still suck :-). JRA.
-                        */
-
-                       if (smb_read_error == 0)
-                               smb_read_error = READ_ERROR;
-                       return -1;
-               }
+               return NT_STATUS_INVALID_PARAMETER;
        }
 
        if(len > 0) {
@@ -1026,21 +780,11 @@ ssize_t receive_smb_raw(int fd,
                        len = MIN(len,maxlen);
                }
 
-               if (timeout > 0) {
-                       ret = read_socket_with_timeout(fd,
-                                       buffer+4,
-                                       len,
-                                       len,
-                                       timeout);
-               } else {
-                       ret = read_data(fd,buffer+4,len);
-               }
+               status = read_socket_with_timeout(
+                       fd, buffer+4, len, len, timeout, &len);
 
-               if (ret != len) {
-                       if (smb_read_error == 0) {
-                               smb_read_error = READ_ERROR;
-                       }
-                       return -1;
+               if (!NT_STATUS_IS_OK(status)) {
+                       return status;
                }
 
                /* not all of samba3 properly checks for packet-termination
@@ -1049,163 +793,8 @@ ssize_t receive_smb_raw(int fd,
                SSVAL(buffer+4,len, 0);
        }
 
-       return len;
-}
-
-static ssize_t receive_smb_raw_talloc(TALLOC_CTX *mem_ctx, int fd,
-                                     char **buffer, unsigned int timeout)
-{
-       char lenbuf[4];
-       ssize_t len,ret;
-
-       smb_read_error = 0;
-
-       len = read_smb_length_return_keepalive(fd, lenbuf, timeout);
-       if (len < 0) {
-               DEBUG(10,("receive_smb_raw: length < 0!\n"));
-
-               /*
-                * Correct fix. smb_read_error may have already been
-                * set. Only set it here if not already set. Global
-                * variables still suck :-). JRA.
-                */
-
-               if (smb_read_error == 0)
-                       smb_read_error = READ_ERROR;
-               return -1;
-       }
-
-       /*
-        * A WRITEX with CAP_LARGE_WRITEX can be 64k worth of data plus 65 bytes
-        * of header. Don't print the error if this fits.... JRA.
-        */
-
-       if (len > (BUFFER_SIZE + LARGE_WRITEX_HDR_SIZE)) {
-               DEBUG(0,("Invalid packet length! (%lu bytes).\n",
-                                       (unsigned long)len));
-               if (len > BUFFER_SIZE + (SAFETY_MARGIN/2)) {
-
-                       /*
-                        * Correct fix. smb_read_error may have already been
-                        * set. Only set it here if not already set. Global
-                        * variables still suck :-). JRA.
-                        */
-
-                       if (smb_read_error == 0)
-                               smb_read_error = READ_ERROR;
-                       return -1;
-               }
-       }
-
-       /*
-        * The +4 here can't wrap, we've checked the length above already.
-        */
-
-       *buffer = TALLOC_ARRAY(mem_ctx, char, len+4);
-
-       if (*buffer == NULL) {
-               DEBUG(0, ("Could not allocate inbuf of length %d\n",
-                         (int)len+4));
-               if (smb_read_error == 0)
-                       smb_read_error = READ_ERROR;
-               return -1;
-       }
-
-       memcpy(*buffer, lenbuf, sizeof(lenbuf));
-
-       if(len > 0) {
-               if (timeout > 0) {
-                       ret = read_socket_with_timeout(fd,(*buffer)+4, len,
-                                                      len, timeout);
-               } else {
-                       ret = read_data(fd, (*buffer)+4, len);
-               }
-
-               if (ret != len) {
-                       if (smb_read_error == 0) {
-                               smb_read_error = READ_ERROR;
-                       }
-                       return -1;
-               }
-       }
-
-       return len + 4;
-}
-
-/****************************************************************************
- Wrapper for receive_smb_raw().
- Checks the MAC on signed packets.
-****************************************************************************/
-
-bool receive_smb(int fd, char *buffer, unsigned int timeout)
-{
-       if (receive_smb_raw(fd, buffer, timeout, 0) < 0) {
-               return false;
-       }
-
-       /* Check the incoming SMB signature. */
-       if (!srv_check_sign_mac(buffer, true)) {
-               DEBUG(0, ("receive_smb: SMB Signature verification "
-                       "failed on incoming packet!\n"));
-               if (smb_read_error == 0) {
-                       smb_read_error = READ_BAD_SIG;
-               }
-               return false;
-       }
-
-       return true;
-}
-
-ssize_t receive_smb_talloc(TALLOC_CTX *mem_ctx, int fd, char **buffer,
-                          unsigned int timeout)
-{
-       ssize_t len;
-
-       len = receive_smb_raw_talloc(mem_ctx, fd, buffer, timeout);
-
-       if (len < 0) {
-               return -1;
-       }
-
-       /* Check the incoming SMB signature. */
-       if (!srv_check_sign_mac(*buffer, true)) {
-               DEBUG(0, ("receive_smb: SMB Signature verification failed on "
-                         "incoming packet!\n"));
-               if (smb_read_error == 0) {
-                       smb_read_error = READ_BAD_SIG;
-               }
-               return -1;
-       }
-
-       return len;
-}
-
-/****************************************************************************
- Send an smb to a fd.
-****************************************************************************/
-
-bool send_smb(int fd, char *buffer)
-{
-       size_t len;
-       size_t nwritten=0;
-       ssize_t ret;
-
-       /* Sign the outgoing packet if required. */
-       srv_calculate_sign_mac(buffer);
-
-       len = smb_len(buffer) + 4;
-
-       while (nwritten < len) {
-               ret = write_data(fd,buffer+nwritten,len - nwritten);
-               if (ret <= 0) {
-                       DEBUG(0,("Error writing %d bytes to client. %d. (%s)\n",
-                               (int)len,(int)ret, strerror(errno) ));
-                       return false;
-               }
-               nwritten += ret;
-       }
-
-       return true;
+       *p_len = len;
+       return NT_STATUS_OK;
 }
 
 /****************************************************************************
@@ -1213,24 +802,28 @@ bool send_smb(int fd, char *buffer)
 ****************************************************************************/
 
 int open_socket_in(int type,
-               int port,
+               uint16_t port,
                int dlevel,
-               uint32 socket_addr,
-               bool rebind )
+               const struct sockaddr_storage *psock,
+               bool rebind)
 {
-       struct sockaddr_in sock;
+       struct sockaddr_storage sock;
        int res;
+       socklen_t slen = sizeof(struct sockaddr_in);
 
-       memset( (char *)&sock, '\0', sizeof(sock) );
+       sock = *psock;
 
-#ifdef HAVE_SOCK_SIN_LEN
-       sock.sin_len         = sizeof(sock);
+#if defined(HAVE_IPV6)
+       if (sock.ss_family == AF_INET6) {
+               ((struct sockaddr_in6 *)&sock)->sin6_port = htons(port);
+               slen = sizeof(struct sockaddr_in6);
+       }
 #endif
-       sock.sin_port        = htons( port );
-       sock.sin_family      = AF_INET;
-       sock.sin_addr.s_addr = socket_addr;
+       if (sock.ss_family == AF_INET) {
+               ((struct sockaddr_in *)&sock)->sin_port = htons(port);
+       }
 
-       res = socket( AF_INET, type, 0 );
+       res = socket(sock.ss_family, type, 0 );
        if( res == -1 ) {
                if( DEBUGLVL(0) ) {
                        dbgtext( "open_socket_in(): socket() call failed: " );
@@ -1258,29 +851,30 @@ int open_socket_in(int type,
                        if( DEBUGLVL( dlevel ) ) {
                                dbgtext( "open_socket_in(): setsockopt: ");
                                dbgtext( "SO_REUSEPORT = %s ",
-                                               val?"true":"false" );
-                               dbgtext( "on port %d failed ", port );
-                               dbgtext( "with error = %s\n", strerror(errno) );
+                                               val?"true":"false");
+                               dbgtext( "on port %d failed ", port);
+                               dbgtext( "with error = %s\n", strerror(errno));
                        }
                }
 #endif /* SO_REUSEPORT */
        }
 
        /* now we've got a socket - we need to bind it */
-       if( bind( res, (struct sockaddr *)&sock, sizeof(sock) ) == -1 ) {
+       if (bind(res, (struct sockaddr *)&sock, slen) == -1 ) {
                if( DEBUGLVL(dlevel) && (port == SMB_PORT1 ||
                                port == SMB_PORT2 || port == NMB_PORT) ) {
-                       dbgtext( "bind failed on port %d ", port );
-                       dbgtext( "socket_addr = %s.\n",
-                                       inet_ntoa( sock.sin_addr ) );
-                       dbgtext( "Error = %s\n", strerror(errno) );
+                       char addr[INET6_ADDRSTRLEN];
+                       print_sockaddr(addr, sizeof(addr),
+                                       &sock);
+                       dbgtext( "bind failed on port %d ", port);
+                       dbgtext( "socket_addr = %s.\n", addr);
+                       dbgtext( "Error = %s\n", strerror(errno));
                }
-               close( res );
+               close(res);
                return -1;
        }
 
        DEBUG( 10, ( "bind succeeded on port %d\n", port ) );
-
        return( res );
  }
 
@@ -1288,38 +882,55 @@ int open_socket_in(int type,
  Create an outgoing socket. timeout is in milliseconds.
 **************************************************************************/
 
-int open_socket_out(int type, struct in_addr *addr, int port ,int timeout)
+int open_socket_out(int type,
+               const struct sockaddr_storage *pss,
+               uint16_t port,
+               int timeout)
 {
-       struct sockaddr_in sock_out;
+       char addr[INET6_ADDRSTRLEN];
+       struct sockaddr_storage sock_out = *pss;
        int res,ret;
        int connect_loop = 10;
        int increment = 10;
 
        /* create a socket to write to */
-       res = socket(PF_INET, type, 0);
+       res = socket(pss->ss_family, type, 0);
        if (res == -1) {
                 DEBUG(0,("socket error (%s)\n", strerror(errno)));
                return -1;
        }
 
-       if (type != SOCK_STREAM)
-               return(res);
-
-       memset((char *)&sock_out,'\0',sizeof(sock_out));
-       putip((char *)&sock_out.sin_addr,(char *)addr);
+       if (type != SOCK_STREAM) {
+               return res;
+       }
 
-       sock_out.sin_port = htons( port );
-       sock_out.sin_family = PF_INET;
+#if defined(HAVE_IPV6)
+       if (pss->ss_family == AF_INET6) {
+               struct sockaddr_in6 *psa6 = (struct sockaddr_in6 *)&sock_out;
+               psa6->sin6_port = htons(port);
+               if (psa6->sin6_scope_id == 0 &&
+                               IN6_IS_ADDR_LINKLOCAL(&psa6->sin6_addr)) {
+                       setup_linklocal_scope_id((struct sockaddr *)&sock_out);
+               }
+       }
+#endif
+       if (pss->ss_family == AF_INET) {
+               struct sockaddr_in *psa = (struct sockaddr_in *)&sock_out;
+               psa->sin_port = htons(port);
+       }
 
        /* set it non-blocking */
        set_blocking(res,false);
 
-       DEBUG(3,("Connecting to %s at port %d\n",inet_ntoa(*addr),port));
+       print_sockaddr(addr, sizeof(addr), &sock_out);
+       DEBUG(3,("Connecting to %s at port %u\n",
+                               addr,
+                               (unsigned int)port));
 
        /* and connect it to the destination */
   connect_again:
 
-       ret = connect(res,(struct sockaddr *)&sock_out,sizeof(sock_out));
+       ret = sys_connect(res, (struct sockaddr *)&sock_out);
 
        /* Some systems return EAGAIN when they mean EINPROGRESS */
        if (ret < 0 && (errno == EINPROGRESS || errno == EALREADY ||
@@ -1336,8 +947,9 @@ int open_socket_out(int type, struct in_addr *addr, int port ,int timeout)
 
        if (ret < 0 && (errno == EINPROGRESS || errno == EALREADY ||
                        errno == EAGAIN)) {
-               DEBUG(1,("timeout connecting to %s:%d\n",
-                                       inet_ntoa(*addr),port));
+               DEBUG(1,("timeout connecting to %s:%u\n",
+                                       addr,
+                                       (unsigned int)port));
                close(res);
                return -1;
        }
@@ -1351,7 +963,9 @@ int open_socket_out(int type, struct in_addr *addr, int port ,int timeout)
 
        if (ret < 0) {
                DEBUG(2,("error connecting to %s:%d (%s)\n",
-                               inet_ntoa(*addr),port,strerror(errno)));
+                               addr,
+                               (unsigned int)port,
+                               strerror(errno)));
                close(res);
                return -1;
        }
@@ -1362,13 +976,24 @@ int open_socket_out(int type, struct in_addr *addr, int port ,int timeout)
        return res;
 }
 
-/****************************************************************************
- Create an outgoing TCP socket to any of the addrs. This is for
- simultaneous connects to port 445 and 139 of a host or even a variety
- of DC's all of which are equivalent for our purposes.
-**************************************************************************/
-
-bool open_any_socket_out(struct sockaddr_in *addrs, int num_addrs,
+/*******************************************************************
+ Create an outgoing TCP socket to the first addr that connects.
+
+ This is for simultaneous connection attempts to port 445 and 139 of a host
+ or for simultatneous connection attempts to multiple DCs at once.  We return
+ a socket fd of the first successful connection.
+
+ @param[in] addrs list of Internet addresses and ports to connect to
+ @param[in] num_addrs number of address/port pairs in the addrs list
+ @param[in] timeout time after which we stop waiting for a socket connection
+            to succeed, given in milliseconds
+ @param[out] fd_index the entry in addrs which we successfully connected to
+ @param[out] fd fd of the open and connected socket
+ @return true on a successful connection, false if all connection attempts
+         failed or we timed out
+*******************************************************************/
+
+bool open_any_socket_out(struct sockaddr_storage *addrs, int num_addrs,
                         int timeout, int *fd_index, int *fd)
 {
        int i, resulting_index, res;
@@ -1394,7 +1019,7 @@ bool open_any_socket_out(struct sockaddr_in *addrs, int num_addrs,
                sockets[i] = -1;
 
        for (i=0; i<num_addrs; i++) {
-               sockets[i] = socket(PF_INET, SOCK_STREAM, 0);
+               sockets[i] = socket(addrs[i].ss_family, SOCK_STREAM, 0);
                if (sockets[i] < 0)
                        goto done;
                set_blocking(sockets[i], false);
@@ -1404,12 +1029,13 @@ bool open_any_socket_out(struct sockaddr_in *addrs, int num_addrs,
        good_connect = false;
 
        for (i=0; i<num_addrs; i++) {
+               const struct sockaddr * a = 
+                   (const struct sockaddr *)&(addrs[i]);
 
                if (sockets[i] == -1)
                        continue;
 
-               if (connect(sockets[i], (struct sockaddr *)&(addrs[i]),
-                           sizeof(*addrs)) == 0) {
+               if (sys_connect(sockets[i], a) == 0) {
                        /* Rather unlikely as we are non-blocking, but it
                         * might actually happen. */
                        resulting_index = i;
@@ -1524,7 +1150,7 @@ int open_udp_socket(const char *host, int port)
        int type = SOCK_DGRAM;
        struct sockaddr_in sock_out;
        int res;
-       struct in_addr *addr;
+       struct in_addr addr;
 
        addr = interpret_addr2(host);
 
@@ -1534,11 +1160,11 @@ int open_udp_socket(const char *host, int port)
        }
 
        memset((char *)&sock_out,'\0',sizeof(sock_out));
-       putip((char *)&sock_out.sin_addr,(char *)addr);
+       putip((char *)&sock_out.sin_addr,(char *)&addr);
        sock_out.sin_port = htons(port);
        sock_out.sin_family = PF_INET;
 
-       if (connect(res,(struct sockaddr *)&sock_out,sizeof(sock_out))) {
+       if (sys_connect(res,(struct sockaddr *)&sock_out)) {
                close(res);
                return -1;
        }
@@ -1552,23 +1178,22 @@ int open_udp_socket(const char *host, int port)
  ******************************************************************/
 
 static const char *get_peer_addr_internal(int fd,
-                               struct sockaddr_storage *pss,
+                               char *addr_buf,
+                               size_t addr_buf_len,
+                               struct sockaddr *pss,
                                socklen_t *plength)
 {
        struct sockaddr_storage ss;
        socklen_t length = sizeof(ss);
-       static char addr_buf[INET6_ADDRSTRLEN];
 
-       safe_strcpy(addr_buf,"0.0.0.0",sizeof(addr_buf)-1);
+       strlcpy(addr_buf,"0.0.0.0",addr_buf_len);
 
        if (fd == -1) {
                return addr_buf;
        }
 
        if (pss == NULL) {
-               pss = &ss;
-       }
-       if (plength == NULL) {
+               pss = (struct sockaddr *)&ss;
                plength = &length;
        }
 
@@ -1578,39 +1203,30 @@ static const char *get_peer_addr_internal(int fd,
                return addr_buf;
        }
 
-       print_sockaddr(addr_buf,
-                       sizeof(addr_buf),
+       print_sockaddr_len(addr_buf,
+                       addr_buf_len,
                        pss,
                        *plength);
        return addr_buf;
 }
 
-
 /*******************************************************************
  Matchname - determine if host name matches IP address. Used to
  confirm a hostname lookup to prevent spoof attacks.
 ******************************************************************/
 
 static bool matchname(const char *remotehost,
-               const struct sockaddr_storage *pss,
+               const struct sockaddr *pss,
                socklen_t len)
 {
-       struct addrinfo hints;
        struct addrinfo *res = NULL;
        struct addrinfo *ailist = NULL;
        char addr_buf[INET6_ADDRSTRLEN];
-       int ret = -1;
-
-       memset(&hints,'\0',sizeof(struct addrinfo));
-       /* By default make sure it supports TCP. */
-       hints.ai_socktype = SOCK_STREAM;
-       hints.ai_flags = AI_ADDRCONFIG|AI_CANONNAME;
-
-       ret = getaddrinfo(remotehost, NULL,
-                       &hints,
-                       &res);
+       bool ret = interpret_string_addr_internal(&ailist,
+                       remotehost,
+                       AI_ADDRCONFIG|AI_CANONNAME);
 
-       if (ret || res == NULL) {
+       if (!ret || ailist == NULL) {
                DEBUG(3,("matchname: getaddrinfo failed for "
                        "name %s [%s]\n",
                        remotehost,
@@ -1622,24 +1238,25 @@ static bool matchname(const char *remotehost,
         * Make sure that getaddrinfo() returns the "correct" host name.
         */
 
-       if (res->ai_canonname == NULL ||
-               (!strequal(remotehost, res->ai_canonname) &&
+       if (ailist->ai_canonname == NULL ||
+               (!strequal(remotehost, ailist->ai_canonname) &&
                 !strequal(remotehost, "localhost"))) {
                DEBUG(0,("matchname: host name/name mismatch: %s != %s\n",
                         remotehost,
-                        res->ai_canonname ? res->ai_canonname : "(NULL)"));
-               freeaddrinfo(res);
+                        ailist->ai_canonname ?
+                                ailist->ai_canonname : "(NULL)"));
+               freeaddrinfo(ailist);
                return false;
        }
 
        /* Look up the host address in the address list we just got. */
-       for (ailist = res; ailist; ailist = ailist->ai_next) {
-               if (!ailist->ai_addr) {
+       for (res = ailist; res; res = res->ai_next) {
+               if (!res->ai_addr) {
                        continue;
                }
-               if (addr_equal((const struct sockaddr_storage *)ailist->ai_addr,
-                                       pss)) {
-                       freeaddrinfo(res);
+               if (sockaddr_equal((const struct sockaddr *)res->ai_addr,
+                                       (struct sockaddr *)pss)) {
+                       freeaddrinfo(ailist);
                        return true;
                }
        }
@@ -1651,71 +1268,133 @@ static bool matchname(const char *remotehost,
         */
 
        DEBUG(0,("matchname: host name/address mismatch: %s != %s\n",
-               print_sockaddr(addr_buf,
+               print_sockaddr_len(addr_buf,
                        sizeof(addr_buf),
                        pss,
                        len),
-                res->ai_canonname ? res->ai_canonname : "(NULL)"));
+                ailist->ai_canonname ? ailist->ai_canonname : "(NULL)"));
 
-       freeaddrinfo(res);
+       if (ailist) {
+               freeaddrinfo(ailist);
+       }
        return false;
 }
 
+/*******************************************************************
+ Deal with the singleton cache.
+******************************************************************/
+
+struct name_addr_pair {
+       struct sockaddr_storage ss;
+       const char *name;
+};
+
+/*******************************************************************
+ Lookup a name/addr pair. Returns memory allocated from memcache.
+******************************************************************/
+
+static bool lookup_nc(struct name_addr_pair *nc)
+{
+       DATA_BLOB tmp;
+
+       ZERO_STRUCTP(nc);
+
+       if (!memcache_lookup(
+                       NULL, SINGLETON_CACHE,
+                       data_blob_string_const_null("get_peer_name"),
+                       &tmp)) {
+               return false;
+       }
+
+       memcpy(&nc->ss, tmp.data, sizeof(nc->ss));
+       nc->name = (const char *)tmp.data + sizeof(nc->ss);
+       return true;
+}
+
+/*******************************************************************
+ Save a name/addr pair.
+******************************************************************/
+
+static void store_nc(const struct name_addr_pair *nc)
+{
+       DATA_BLOB tmp;
+       size_t namelen = strlen(nc->name);
+
+       tmp = data_blob(NULL, sizeof(nc->ss) + namelen + 1);
+       if (!tmp.data) {
+               return;
+       }
+       memcpy(tmp.data, &nc->ss, sizeof(nc->ss));
+       memcpy(tmp.data+sizeof(nc->ss), nc->name, namelen+1);
+
+       memcache_add(NULL, SINGLETON_CACHE,
+                       data_blob_string_const_null("get_peer_name"),
+                       tmp);
+       data_blob_free(&tmp);
+}
+
 /*******************************************************************
  Return the DNS name of the remote end of a socket.
 ******************************************************************/
 
 const char *get_peer_name(int fd, bool force_lookup)
 {
-       static fstring addr_buf;
-       static pstring name_buf;
+       struct name_addr_pair nc;
+       char addr_buf[INET6_ADDRSTRLEN];
        struct sockaddr_storage ss;
        socklen_t length = sizeof(ss);
        const char *p;
        int ret;
-       pstring tmp_name;
+       char name_buf[MAX_DNS_NAME_LENGTH];
+       char tmp_name[MAX_DNS_NAME_LENGTH];
 
        /* reverse lookups can be *very* expensive, and in many
           situations won't work because many networks don't link dhcp
           with dns. To avoid the delay we avoid the lookup if
           possible */
        if (!lp_hostname_lookups() && (force_lookup == false)) {
-               return get_peer_addr(fd);
+               length = sizeof(nc.ss);
+               nc.name = get_peer_addr_internal(fd, addr_buf, sizeof(addr_buf),
+                       (struct sockaddr *)&nc.ss, &length);
+               store_nc(&nc);
+               lookup_nc(&nc);
+               return nc.name ? nc.name : "UNKNOWN";
        }
 
-       p = get_peer_addr_internal(fd, &ss, &length);
+       lookup_nc(&nc);
+
+       memset(&ss, '\0', sizeof(ss));
+       p = get_peer_addr_internal(fd, addr_buf, sizeof(addr_buf), (struct sockaddr *)&ss, &length);
 
        /* it might be the same as the last one - save some DNS work */
-       if (strcmp(p, addr_buf) == 0) {
-               return name_buf;
+       if (sockaddr_equal((struct sockaddr *)&ss, (struct sockaddr *)&nc.ss)) {
+               return nc.name ? nc.name : "UNKNOWN";
        }
 
-       pstrcpy(name_buf,"UNKNOWN");
+       /* Not the same. We need to lookup. */
        if (fd == -1) {
-               return name_buf;
+               return "UNKNOWN";
        }
 
-       fstrcpy(addr_buf, p);
-
        /* Look up the remote host name. */
-       ret = getnameinfo((struct sockaddr *)&ss,
+       ret = sys_getnameinfo((struct sockaddr *)&ss,
                        length,
                        name_buf,
                        sizeof(name_buf),
                        NULL,
                        0,
-                       NI_NUMERICHOST);
+                       0);
 
        if (ret) {
                DEBUG(1,("get_peer_name: getnameinfo failed "
                        "for %s with error %s\n",
                        p,
                        gai_strerror(ret)));
-               pstrcpy(name_buf, p);
+               strlcpy(name_buf, p, sizeof(name_buf));
        } else {
-               if (!matchname(name_buf, &ss, length)) {
+               if (!matchname(name_buf, (struct sockaddr *)&ss, length)) {
                        DEBUG(0,("Matchname failed on %s %s\n",name_buf,p));
-                       pstrcpy(name_buf,"UNKNOWN");
+                       strlcpy(name_buf,"UNKNOWN",sizeof(name_buf));
                }
        }
 
@@ -1723,22 +1402,27 @@ const char *get_peer_name(int fd, bool force_lookup)
           use --enable-developer or the clobber_region() call will
           get you */
 
-       pstrcpy(tmp_name, name_buf );
+       strlcpy(tmp_name, name_buf, sizeof(tmp_name));
        alpha_strcpy(name_buf, tmp_name, "_-.", sizeof(name_buf));
        if (strstr(name_buf,"..")) {
-               pstrcpy(name_buf, "UNKNOWN");
+               strlcpy(name_buf, "UNKNOWN", sizeof(name_buf));
        }
 
-       return name_buf;
+       nc.name = name_buf;
+       nc.ss = ss;
+
+       store_nc(&nc);
+       lookup_nc(&nc);
+       return nc.name ? nc.name : "UNKNOWN";
 }
 
 /*******************************************************************
  Return the IP addr of the remote end of a socket as a string.
  ******************************************************************/
 
-const char *get_peer_addr(int fd)
+const char *get_peer_addr(int fd, char *addr, size_t addr_len)
 {
-       return get_peer_addr_internal(fd, NULL, NULL);
+       return get_peer_addr_internal(fd, addr, addr_len, NULL, NULL);
 }
 
 /*******************************************************************
@@ -1758,7 +1442,7 @@ int create_pipe_sock(const char *socket_dir,
        struct stat st;
        int sock;
        mode_t old_umask;
-       pstring path;
+       char *path = NULL;
 
        old_umask = umask(0);
 
@@ -1798,16 +1482,19 @@ int create_pipe_sock(const char *socket_dir,
        sock = socket(AF_UNIX, SOCK_STREAM, 0);
 
        if (sock == -1) {
-               perror("socket");
-                goto out_umask;
+               DEBUG(0, ("create_pipe_sock: socket error %s\n",
+                       strerror(errno) ));
+                goto out_close;
        }
 
-       pstr_sprintf(path, "%s/%s", socket_dir, socket_name);
+       if (asprintf(&path, "%s/%s", socket_dir, socket_name) == -1) {
+                goto out_close;
+       }
 
        unlink(path);
        memset(&sunaddr, 0, sizeof(sunaddr));
        sunaddr.sun_family = AF_UNIX;
-       safe_strcpy(sunaddr.sun_path, path, sizeof(sunaddr.sun_path)-1);
+       strlcpy(sunaddr.sun_path, path, sizeof(sunaddr.sun_path));
 
        if (bind(sock, (struct sockaddr *)&sunaddr, sizeof(sunaddr)) == -1) {
                DEBUG(0, ("bind failed on pipe socket %s: %s\n", path,
@@ -1821,11 +1508,15 @@ int create_pipe_sock(const char *socket_dir,
                goto out_close;
        }
 
+       SAFE_FREE(path);
+
        umask(old_umask);
        return sock;
 
 out_close:
-       close(sock);
+       SAFE_FREE(path);
+       if (sock != -1)
+               close(sock);
 
 out_umask:
        umask(old_umask);
@@ -1837,91 +1528,173 @@ out_umask:
 #endif /* HAVE_UNIXSOCKET */
 }
 
+/****************************************************************************
+ Get my own canonical name, including domain.
+****************************************************************************/
+
+const char *get_mydnsfullname(void)
+{
+       struct addrinfo *res = NULL;
+       char my_hostname[HOST_NAME_MAX];
+       bool ret;
+       DATA_BLOB tmp;
+
+       if (memcache_lookup(NULL, SINGLETON_CACHE,
+                       data_blob_string_const_null("get_mydnsfullname"),
+                       &tmp)) {
+               SMB_ASSERT(tmp.length > 0);
+               return (const char *)tmp.data;
+       }
+
+       /* get my host name */
+       if (gethostname(my_hostname, sizeof(my_hostname)) == -1) {
+               DEBUG(0,("get_mydnsfullname: gethostname failed\n"));
+               return NULL;
+       }
+
+       /* Ensure null termination. */
+       my_hostname[sizeof(my_hostname)-1] = '\0';
+
+       ret = interpret_string_addr_internal(&res,
+                               my_hostname,
+                               AI_ADDRCONFIG|AI_CANONNAME);
+
+       if (!ret || res == NULL) {
+               DEBUG(3,("get_mydnsfullname: getaddrinfo failed for "
+                       "name %s [%s]\n",
+                       my_hostname,
+                       gai_strerror(ret) ));
+               return NULL;
+       }
+
+       /*
+        * Make sure that getaddrinfo() returns the "correct" host name.
+        */
+
+       if (res->ai_canonname == NULL) {
+               DEBUG(3,("get_mydnsfullname: failed to get "
+                       "canonical name for %s\n",
+                       my_hostname));
+               freeaddrinfo(res);
+               return NULL;
+       }
+
+       /* This copies the data, so we must do a lookup
+        * afterwards to find the value to return.
+        */
+
+       memcache_add(NULL, SINGLETON_CACHE,
+                       data_blob_string_const_null("get_mydnsfullname"),
+                       data_blob_string_const_null(res->ai_canonname));
+
+       if (!memcache_lookup(NULL, SINGLETON_CACHE,
+                       data_blob_string_const_null("get_mydnsfullname"),
+                       &tmp)) {
+               tmp = data_blob_talloc(talloc_tos(), res->ai_canonname,
+                               strlen(res->ai_canonname) + 1);
+       }
+
+       freeaddrinfo(res);
+
+       return (const char *)tmp.data;
+}
+
 /************************************************************
- Is this my name ? Needs fixing for IPv6.
+ Is this my name ?
 ************************************************************/
 
 bool is_myname_or_ipaddr(const char *s)
 {
-       fstring name, dnsname;
-       char *servername;
+       TALLOC_CTX *ctx = talloc_tos();
+       char *name = NULL;
+       const char *dnsname;
+       char *servername = NULL;
 
-       if ( !s )
+       if (!s) {
                return false;
+       }
 
-       /* santize the string from '\\name' */
-
-       fstrcpy( name, s );
+       /* Santize the string from '\\name' */
+       name = talloc_strdup(ctx, s);
+       if (!name) {
+               return false;
+       }
 
-       servername = strrchr_m( name, '\\' );
-       if ( !servername )
+       servername = strrchr_m(name, '\\' );
+       if (!servername) {
                servername = name;
-       else
+       } else {
                servername++;
+       }
 
-       /* optimize for the common case */
-
-       if (strequal(servername, global_myname()))
+       /* Optimize for the common case */
+       if (strequal(servername, global_myname())) {
                return true;
+       }
 
-       /* check for an alias */
-
-       if (is_myname(servername))
+       /* Check for an alias */
+       if (is_myname(servername)) {
                return true;
+       }
 
-       /* check for loopback */
-
-       if (strequal(servername, "127.0.0.1"))
+       /* Check for loopback */
+       if (strequal(servername, "127.0.0.1") ||
+                       strequal(servername, "::1")) {
                return true;
+       }
 
-       if (strequal(servername, "localhost"))
+       if (strequal(servername, "localhost")) {
                return true;
+       }
 
-       /* maybe it's my dns name */
-
-       if ( get_mydnsfullname( dnsname ) )
-               if ( strequal( servername, dnsname ) )
-                       return true;
-
-       /* handle possible CNAME records */
-
-       if ( !is_ipaddress_v4( servername ) ) {
-               /* use DNS to resolve the name, but only the first address */
-               struct hostent *hp;
+       /* Maybe it's my dns name */
+       dnsname = get_mydnsfullname();
+       if (dnsname && strequal(servername, dnsname)) {
+               return true;
+       }
 
-               if (((hp = sys_gethostbyname(name)) != NULL) && (hp->h_addr != NULL)) {
-                       struct in_addr return_ip;
-                       putip( (char*)&return_ip, (char*)hp->h_addr );
-                       fstrcpy( name, inet_ntoa( return_ip ) );
+       /* Handle possible CNAME records - convert to an IP addr. */
+       if (!is_ipaddress(servername)) {
+               /* Use DNS to resolve the name, but only the first address */
+               struct sockaddr_storage ss;
+               if (interpret_string_addr(&ss, servername,0)) {
+                       print_sockaddr(name,
+                                       sizeof(name),
+                                       &ss);
                        servername = name;
                }
        }
 
-       /* maybe its an IP address? */
-       if (is_ipaddress_v4(servername)) {
+       /* Maybe its an IP address? */
+       if (is_ipaddress(servername)) {
                struct sockaddr_storage ss;
-               struct iface_struct nics[MAX_INTERFACES];
+               struct iface_struct *nics;
                int i, n;
-               struct in_addr ip;
 
-               ip = *interpret_addr2(servername);
-               if (is_zero_ip_v4(ip) || is_loopback_ip_v4(ip)) {
+               if (!interpret_string_addr(&ss, servername, AI_NUMERICHOST)) {
                        return false;
                }
 
-               in_addr_to_sockaddr_storage(&ss, ip);
+               if (is_zero_addr((struct sockaddr *)&ss) || 
+                       is_loopback_addr((struct sockaddr *)&ss)) {
+                       return false;
+               }
 
+               nics = TALLOC_ARRAY(ctx, struct iface_struct,
+                                       MAX_INTERFACES);
+               if (!nics) {
+                       return false;
+               }
                n = get_interfaces(nics, MAX_INTERFACES);
                for (i=0; i<n; i++) {
-                       if (nics[i].ip.ss_family != AF_INET) {
-                               continue;
-                       }
-                       if (addr_equal(&nics[i].ip, &ss)) {
+                       if (sockaddr_equal((struct sockaddr *)&nics[i].ip, (struct sockaddr *)&ss)) {
+                               TALLOC_FREE(nics);
                                return true;
                        }
                }
+               TALLOC_FREE(nics);
        }
 
-       /* no match */
+       /* No match */
        return false;
 }