s3: Change sockaddr util function names for consistency
[samba.git] / source3 / lib / util_sock.c
index f62b20b6fcaccd201ebeec5e979a3a69740ee3ef..605bbf1fb6b4b557e6aff4926cc09f196a87276d 100644 (file)
 
 #include "includes.h"
 
-/****************************************************************************
- Return true if a string could be an IPv4 address.
-****************************************************************************/
-
-bool is_ipaddress_v4(const char *str)
-{
-       int ret = -1;
-       struct in_addr dest;
-
-       ret = inet_pton(AF_INET, str, &dest);
-       if (ret > 0) {
-               return true;
-       }
-       return false;
-}
-
-/****************************************************************************
- Return true if a string could be an IPv4 or IPv6 address.
-****************************************************************************/
-
-bool is_ipaddress(const char *str)
-{
-       int ret = -1;
-
-#if defined(HAVE_IPV6)
-       if (strchr_m(str, ':')) {
-               char addr[INET6_ADDRSTRLEN];
-               struct in6_addr dest6;
-               const char *sp = str;
-               char *p = strchr_m(str, '%');
-
-               /*
-                * Cope with link-local.
-                * This is IP:v6:addr%ifname.
-                */
-
-               if (p && (p > str) && (if_nametoindex(p+1) != 0)) {
-                       strlcpy(addr, str,
-                               MIN(PTR_DIFF(p,str)+1,
-                                       sizeof(addr)));
-                       sp = addr;
-               }
-               ret = inet_pton(AF_INET6, addr, &dest6);
-               if (ret > 0) {
-                       return true;
-               }
-       }
-#endif
-       return is_ipaddress_v4(str);
-}
-
-/****************************************************************************
- Is a sockaddr_storage a broadcast address ?
-****************************************************************************/
-
-bool is_broadcast_addr(const struct sockaddr_storage *pss)
-{
-#if defined(HAVE_IPV6)
-       if (pss->ss_family == AF_INET6) {
-               const struct in6_addr *sin6 =
-                       &((const struct sockaddr_in6 *)pss)->sin6_addr;
-               return IN6_IS_ADDR_MULTICAST(sin6);
-       }
-#endif
-       if (pss->ss_family == AF_INET) {
-               uint32_t addr =
-               ntohl(((const struct sockaddr_in *)pss)->sin_addr.s_addr);
-               return addr == INADDR_BROADCAST;
-       }
-       return false;
-}
-
-/*******************************************************************
- Wrap getaddrinfo...
-******************************************************************/
-
-static bool interpret_string_addr_internal(struct addrinfo **ppres,
-                                       const char *str, int flags)
-{
-       int ret;
-       struct addrinfo hints;
-
-       memset(&hints, '\0', sizeof(hints));
-       /* By default make sure it supports TCP. */
-       hints.ai_socktype = SOCK_STREAM;
-       hints.ai_flags = flags;
-
-       ret = getaddrinfo(str, NULL,
-                       &hints,
-                       ppres);
-       if (ret) {
-               DEBUG(3,("interpret_string_addr_internal: getaddrinfo failed "
-                       "for name %s [%s]\n",
-                       str,
-                       gai_strerror(ret) ));
-               return false;
-       }
-       return true;
-}
-
-/****************************************************************************
- Interpret an internet address or name into an IP address in 4 byte form.
- RETURNS IN NETWORK BYTE ORDER (big endian).
-****************************************************************************/
-
-uint32 interpret_addr(const char *str)
-{
-       uint32 ret;
-
-       /* If it's in the form of an IP address then
-        * get the lib to interpret it */
-       if (is_ipaddress_v4(str)) {
-               struct in_addr dest;
-
-               if (inet_pton(AF_INET, str, &dest) <= 0) {
-                       /* Error - this shouldn't happen ! */
-                       DEBUG(0,("interpret_addr: inet_pton failed "
-                               "host %s\n",
-                               str));
-                       return 0;
-               }
-               ret = dest.s_addr; /* NETWORK BYTE ORDER ! */
-       } else {
-               /* Otherwise assume it's a network name of some sort and use
-                       getadddrinfo. */
-               struct addrinfo *res = NULL;
-               struct addrinfo *res_list = NULL;
-               if (!interpret_string_addr_internal(&res_list,
-                                       str,
-                                       AI_ADDRCONFIG)) {
-                       DEBUG(3,("interpret_addr: Unknown host. %s\n",str));
-                       return 0;
-               }
-
-               /* Find the first IPv4 address. */
-               for (res = res_list; res; res = res->ai_next) {
-                       if (res->ai_family != AF_INET) {
-                               continue;
-                       }
-                       if (res->ai_addr == NULL) {
-                               continue;
-                       }
-                       break;
-               }
-               if(res == NULL) {
-                       DEBUG(3,("interpret_addr: host address is "
-                               "invalid for host %s\n",str));
-                       if (res_list) {
-                               freeaddrinfo(res_list);
-                       }
-                       return 0;
-               }
-               putip((char *)&ret,
-                       &((struct sockaddr_in *)res->ai_addr)->sin_addr.s_addr);
-               if (res_list) {
-                       freeaddrinfo(res_list);
-               }
-       }
-
-       /* This is so bogus - all callers need fixing... JRA. */
-       if (ret == (uint32)-1) {
-               return 0;
-       }
-
-       return ret;
-}
-
-/*******************************************************************
- A convenient addition to interpret_addr().
-******************************************************************/
-
-struct in_addr *interpret_addr2(struct in_addr *ip, const char *str)
-{
-       uint32 a = interpret_addr(str);
-       ip->s_addr = a;
-       return ip;
-}
-
 /*******************************************************************
  Map a text hostname or IP address (IPv4 or IPv6) into a
  struct sockaddr_storage.
@@ -208,9 +30,9 @@ bool interpret_string_addr(struct sockaddr_storage *pss,
                const char *str,
                int flags)
 {
-       char addr[INET6_ADDRSTRLEN];
        struct addrinfo *res = NULL;
 #if defined(HAVE_IPV6)
+       char addr[INET6_ADDRSTRLEN];
        unsigned int scope_id = 0;
 
        if (strchr_m(str, ':')) {
@@ -230,7 +52,7 @@ bool interpret_string_addr(struct sockaddr_storage *pss,
        }
 #endif
 
-       zero_addr(pss);
+       zero_sockaddr(pss);
 
        if (!interpret_string_addr_internal(&res, str, flags|AI_ADDRCONFIG)) {
                return false;
@@ -255,226 +77,17 @@ bool interpret_string_addr(struct sockaddr_storage *pss,
        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.
-******************************************************************/
-
-bool is_loopback_addr(const struct sockaddr_storage *pss)
-{
-#if defined(HAVE_IPV6)
-       if (pss->ss_family == AF_INET6) {
-               struct in6_addr *pin6 =
-                       &((struct sockaddr_in6 *)pss)->sin6_addr;
-               return IN6_IS_ADDR_LOOPBACK(pin6);
-       }
-#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.
-******************************************************************/
-
-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(HAVE_IPV6)
-       if (pss->ss_family == AF_INET6) {
-               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);
-       }
-       return false;
-}
-
-/*******************************************************************
- Set an IP to 0.0.0.0.
-******************************************************************/
-
-void zero_ip_v4(struct in_addr *ip)
-{
-       memset(ip, '\0', sizeof(struct in_addr));
-}
-
 /*******************************************************************
  Set an address to INADDR_ANY.
 ******************************************************************/
 
-void zero_addr(struct sockaddr_storage *pss)
+void zero_sockaddr(struct sockaddr_storage *pss)
 {
        memset(pss, '\0', sizeof(*pss));
        /* Ensure we're at least a valid sockaddr-storage. */
        pss->ss_family = AF_INET;
 }
 
-/*******************************************************************
- 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;
-}
-
-#if defined(HAVE_IPV6)
-/*******************************************************************
- 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. */
-               return false;
-       }
-
-#if defined(HAVE_IPV6)
-       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++;
-               }
-               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;
-}
-
-/*******************************************************************
- Are two sockaddr_storage's the same family and address ? Ignore port etc.
-********************************************************************/
-
-bool addr_equal(const struct sockaddr_storage *ip1,
-               const struct sockaddr_storage *ip2)
-{
-       if (ip1->ss_family != ip2->ss_family) {
-               /* Never the same. */
-               return false;
-       }
-
-#if defined(HAVE_IPV6)
-       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;
-}
-
-/****************************************************************************
- Is an IP address the INADDR_ANY or in6addr_any value ?
-****************************************************************************/
-
-bool is_address_any(const struct sockaddr_storage *psa)
-{
-#if defined(HAVE_IPV6)
-       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;
-       }
-#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;
-       }
-       return false;
-}
-
 /****************************************************************************
  Get a port number in host byte order from a sockaddr_storage.
 ****************************************************************************/
@@ -504,13 +117,13 @@ uint16_t get_sockaddr_port(const struct sockaddr_storage *pss)
 
 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)sys_getnameinfo((const struct sockaddr *)psa,
+       (void)sys_getnameinfo(psa,
                        psalen,
                        dest, destlen,
                        NULL, 0,
@@ -526,7 +139,7 @@ char *print_sockaddr(char *dest,
                        size_t destlen,
                        const struct sockaddr_storage *psa)
 {
-       return print_sockaddr_len(dest, destlen, psa,
+       return print_sockaddr_len(dest, destlen, (struct sockaddr *)psa,
                        sizeof(struct sockaddr_storage));
 }
 
@@ -541,50 +154,28 @@ char *print_canonical_sockaddr(TALLOC_CTX *ctx,
        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) {
+       if (ret != 0) {
                return NULL;
        }
+
        if (pss->ss_family != AF_INET) {
 #if defined(HAVE_IPV6)
-               /* IPv6 */
-               const struct sockaddr_in6 *sa6 =
-                       (const struct sockaddr_in6 *)pss;
-               uint16_t port = ntohs(sa6->sin6_port);
-
-               if (port) {
-                       dest = talloc_asprintf(ctx,
-                                       "[%s]:%d",
-                                       addr,
-                                       (unsigned int)port);
-               } else {
-                       dest = talloc_asprintf(ctx,
-                                       "[%s]",
-                                       addr);
-               }
+               dest = talloc_asprintf(ctx, "[%s]", addr);
 #else
                return NULL;
 #endif
        } else {
-               const struct sockaddr_in *sa =
-                       (const struct sockaddr_in *)pss;
-               uint16_t port = ntohs(sa->sin_port);
-
-               if (port) {
-                       dest = talloc_asprintf(ctx,
-                                       "%s:%d",
-                                       addr,
-                                       (unsigned int)port);
-               } else {
-                       dest = talloc_asprintf(ctx,
-                                       "%s",
-                                       addr);
-               }
+               dest = talloc_asprintf(ctx, "%s", addr);
        }
+       
        return dest;
 }
 
@@ -614,7 +205,7 @@ static const char *get_socket_addr(int fd, char *addr_buf, size_t addr_len)
                return addr_buf;
        }
 
-       return print_sockaddr_len(addr_buf, addr_len, &sa, length);
+       return print_sockaddr_len(addr_buf, addr_len, (struct sockaddr *)&sa, length);
 }
 
 #if 0
@@ -650,18 +241,6 @@ static int get_socket_port(int fd)
 }
 #endif
 
-void set_sockaddr_port(struct sockaddr_storage *psa, uint16 port)
-{
-#if defined(HAVE_IPV6)
-       if (psa->ss_family == AF_INET6) {
-               ((struct sockaddr_in6 *)psa)->sin6_port = htons(port);
-       }
-#endif
-       if (psa->ss_family == AF_INET) {
-               ((struct sockaddr_in *)psa)->sin_port = htons(port);
-       }
-}
-
 const char *client_name(int fd)
 {
        return get_peer_name(fd,false);
@@ -855,7 +434,10 @@ 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) ));
                }
        }
@@ -913,10 +495,10 @@ ssize_t read_udp_v4_socket(int fd,
  time_out = timeout in milliseconds
 ****************************************************************************/
 
-NTSTATUS read_socket_with_timeout_ntstatus(int fd, char *buf,
-                                          size_t mincnt, size_t maxcnt,
-                                          unsigned int time_out,
-                                          size_t *size_ret)
+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;
@@ -1042,44 +624,13 @@ NTSTATUS read_socket_with_timeout_ntstatus(int fd, char *buf,
        return NT_STATUS_OK;
 }
 
-ssize_t read_socket_with_timeout(int fd, char *buf,
-                                size_t mincnt, size_t maxcnt,
-                                unsigned int time_out,
-                                enum smb_read_errors *pre)
-{
-       NTSTATUS status;
-       size_t size_ret;
-
-       set_smb_read_error(pre, SMB_READ_OK);
-
-       status = read_socket_with_timeout_ntstatus(fd, buf, mincnt, maxcnt,
-                                                  time_out, &size_ret);
-
-       if (NT_STATUS_IS_OK(status)) {
-               return size_ret;
-       }
-
-       if (NT_STATUS_EQUAL(status, NT_STATUS_END_OF_FILE)) {
-               set_smb_read_error(pre, SMB_READ_EOF);
-               return -1;
-       }
-
-       if (NT_STATUS_EQUAL(status, NT_STATUS_IO_TIMEOUT)) {
-               set_smb_read_error(pre, SMB_READ_TIMEOUT);
-               return -1;
-       }
-
-       set_smb_read_error(pre, SMB_READ_ERROR);
-       return -1;
-}
-
 /****************************************************************************
  Read data from the client, reading exactly N bytes.
 ****************************************************************************/
 
-ssize_t read_data(int fd,char *buffer,size_t N, enum smb_read_errors *pre)
+NTSTATUS read_data(int fd, char *buffer, size_t N)
 {
-       return read_socket_with_timeout(fd, buffer, N, N, 0, pre);
+       return read_socket_with_timeout(fd, buffer, N, N, 0, NULL);
 }
 
 /****************************************************************************
@@ -1148,8 +699,7 @@ NTSTATUS read_smb_length_return_keepalive(int fd, char *inbuf,
        int msg_type;
        NTSTATUS status;
 
-       status = read_socket_with_timeout_ntstatus(fd, inbuf, 4, 4, timeout,
-                                                  NULL);
+       status = read_socket_with_timeout(fd, inbuf, 4, 4, timeout, NULL);
 
        if (!NT_STATUS_IS_OK(status)) {
                return status;
@@ -1162,7 +712,7 @@ NTSTATUS read_smb_length_return_keepalive(int fd, char *inbuf,
                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 NT_STATUS_OK;
 }
@@ -1174,31 +724,18 @@ NTSTATUS read_smb_length_return_keepalive(int fd, char *inbuf,
  Timeout is in milliseconds.
 ****************************************************************************/
 
-ssize_t read_smb_length(int fd, char *inbuf, unsigned int timeout, enum smb_read_errors *pre)
+NTSTATUS read_smb_length(int fd, char *inbuf, unsigned int timeout,
+                        size_t *len)
 {
-       size_t len;
        uint8_t msgtype = SMBkeepalive;
 
-       set_smb_read_error(pre, SMB_READ_OK);
-
        while (msgtype == SMBkeepalive) {
                NTSTATUS status;
 
                status = read_smb_length_return_keepalive(fd, inbuf, timeout,
-                                                         &len);
+                                                         len);
                if (!NT_STATUS_IS_OK(status)) {
-                       if (NT_STATUS_EQUAL(status, NT_STATUS_END_OF_FILE)) {
-                               set_smb_read_error(pre, SMB_READ_EOF);
-                               return -1;
-                       }
-
-                       if (NT_STATUS_EQUAL(status, NT_STATUS_IO_TIMEOUT)) {
-                               set_smb_read_error(pre, SMB_READ_TIMEOUT);
-                               return -1;
-                       }
-
-                       set_smb_read_error(pre, SMB_READ_ERROR);
-                       return -1;
+                       return status;
                }
 
                msgtype = CVAL(inbuf, 0);
@@ -1207,69 +744,35 @@ ssize_t read_smb_length(int fd, char *inbuf, unsigned int timeout, enum smb_read
        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,
-                       enum smb_read_errors *pre)
+NTSTATUS receive_smb_raw(int fd, char *buffer, size_t buflen, unsigned int timeout,
+                        size_t maxlen, size_t *p_len)
 {
        size_t len;
-       ssize_t ret;
        NTSTATUS status;
 
-       set_smb_read_error(pre,SMB_READ_OK);
-
        status = read_smb_length_return_keepalive(fd,buffer,timeout,&len);
 
        if (!NT_STATUS_IS_OK(status)) {
                DEBUG(10, ("receive_smb_raw: %s!\n", nt_errstr(status)));
-
-               if (NT_STATUS_EQUAL(status, NT_STATUS_END_OF_FILE)) {
-                       set_smb_read_error(pre, SMB_READ_EOF);
-                       return -1;
-               }
-
-               if (NT_STATUS_EQUAL(status, NT_STATUS_IO_TIMEOUT)) {
-                       set_smb_read_error(pre, SMB_READ_TIMEOUT);
-                       return -1;
-               }
-
-               set_smb_read_error(pre, SMB_READ_ERROR);
-               return -1;
+               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.
-                        */
-
-                       cond_set_smb_read_error(pre,SMB_READ_ERROR);
-                       return -1;
-               }
+               return NT_STATUS_INVALID_PARAMETER;
        }
 
        if(len > 0) {
@@ -1277,12 +780,11 @@ ssize_t receive_smb_raw(int fd,
                        len = MIN(len,maxlen);
                }
 
-               ret = read_socket_with_timeout(fd, buffer+4, len, len, timeout,
-                                              pre);
+               status = read_socket_with_timeout(
+                       fd, buffer+4, len, len, timeout, &len);
 
-               if (ret != len) {
-                       cond_set_smb_read_error(pre,SMB_READ_ERROR);
-                       return -1;
+               if (!NT_STATUS_IS_OK(status)) {
+                       return status;
                }
 
                /* not all of samba3 properly checks for packet-termination
@@ -1291,7 +793,8 @@ ssize_t receive_smb_raw(int fd,
                SSVAL(buffer+4,len, 0);
        }
 
-       return len;
+       *p_len = len;
+       return NT_STATUS_OK;
 }
 
 /****************************************************************************
@@ -1407,7 +910,7 @@ int open_socket_out(int type,
                psa6->sin6_port = htons(port);
                if (psa6->sin6_scope_id == 0 &&
                                IN6_IS_ADDR_LINKLOCAL(&psa6->sin6_addr)) {
-                       setup_linklocal_scope_id(&sock_out);
+                       setup_linklocal_scope_id((struct sockaddr *)&sock_out);
                }
        }
 #endif
@@ -1473,11 +976,22 @@ int open_socket_out(int type,
        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.
-**************************************************************************/
+/*******************************************************************
+ 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)
@@ -1638,7 +1152,7 @@ int open_udp_socket(const char *host, int port)
        int res;
        struct in_addr addr;
 
-       (void)interpret_addr2(&addr, host);
+       addr = interpret_addr2(host);
 
        res = socket(PF_INET, type, 0);
        if (res == -1) {
@@ -1666,7 +1180,7 @@ int open_udp_socket(const char *host, int port)
 static const char *get_peer_addr_internal(int fd,
                                char *addr_buf,
                                size_t addr_buf_len,
-                               struct sockaddr_storage *pss,
+                               struct sockaddr *pss,
                                socklen_t *plength)
 {
        struct sockaddr_storage ss;
@@ -1679,9 +1193,7 @@ static const char *get_peer_addr_internal(int fd,
        }
 
        if (pss == NULL) {
-               pss = &ss;
-       }
-       if (plength == NULL) {
+               pss = (struct sockaddr *)&ss;
                plength = &length;
        }
 
@@ -1704,7 +1216,7 @@ static const char *get_peer_addr_internal(int fd,
 ******************************************************************/
 
 static bool matchname(const char *remotehost,
-               const struct sockaddr_storage *pss,
+               const struct sockaddr *pss,
                socklen_t len)
 {
        struct addrinfo *res = NULL;
@@ -1742,8 +1254,8 @@ static bool matchname(const char *remotehost,
                if (!res->ai_addr) {
                        continue;
                }
-               if (addr_equal((const struct sockaddr_storage *)res->ai_addr,
-                                       pss)) {
+               if (sockaddr_equal((const struct sockaddr *)res->ai_addr,
+                                       (struct sockaddr *)pss)) {
                        freeaddrinfo(ailist);
                        return true;
                }
@@ -1789,7 +1301,7 @@ static bool lookup_nc(struct name_addr_pair *nc)
 
        if (!memcache_lookup(
                        NULL, SINGLETON_CACHE,
-                       data_blob_string_const("get_peer_name"),
+                       data_blob_string_const_null("get_peer_name"),
                        &tmp)) {
                return false;
        }
@@ -1816,7 +1328,7 @@ static void store_nc(const struct name_addr_pair *nc)
        memcpy(tmp.data+sizeof(nc->ss), nc->name, namelen+1);
 
        memcache_add(NULL, SINGLETON_CACHE,
-                       data_blob_string_const("get_peer_name"),
+                       data_blob_string_const_null("get_peer_name"),
                        tmp);
        data_blob_free(&tmp);
 }
@@ -1843,7 +1355,7 @@ const char *get_peer_name(int fd, bool force_lookup)
        if (!lp_hostname_lookups() && (force_lookup == false)) {
                length = sizeof(nc.ss);
                nc.name = get_peer_addr_internal(fd, addr_buf, sizeof(addr_buf),
-                       &nc.ss, &length);
+                       (struct sockaddr *)&nc.ss, &length);
                store_nc(&nc);
                lookup_nc(&nc);
                return nc.name ? nc.name : "UNKNOWN";
@@ -1852,10 +1364,10 @@ const char *get_peer_name(int fd, bool force_lookup)
        lookup_nc(&nc);
 
        memset(&ss, '\0', sizeof(ss));
-       p = get_peer_addr_internal(fd, addr_buf, sizeof(addr_buf), &ss, &length);
+       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 (addr_equal(&ss, &nc.ss)) {
+       if (sockaddr_equal((struct sockaddr *)&ss, (struct sockaddr *)&nc.ss)) {
                return nc.name ? nc.name : "UNKNOWN";
        }
 
@@ -1880,7 +1392,7 @@ const char *get_peer_name(int fd, bool force_lookup)
                        gai_strerror(ret)));
                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));
                        strlcpy(name_buf,"UNKNOWN",sizeof(name_buf));
                }
@@ -1975,8 +1487,7 @@ int create_pipe_sock(const char *socket_dir,
                 goto out_close;
        }
 
-       asprintf(&path, "%s/%s", socket_dir, socket_name);
-       if (!path) {
+       if (asprintf(&path, "%s/%s", socket_dir, socket_name) == -1) {
                 goto out_close;
        }
 
@@ -2004,7 +1515,8 @@ int create_pipe_sock(const char *socket_dir,
 
 out_close:
        SAFE_FREE(path);
-       close(sock);
+       if (sock != -1)
+               close(sock);
 
 out_umask:
        umask(old_umask);
@@ -2028,7 +1540,7 @@ const char *get_mydnsfullname(void)
        DATA_BLOB tmp;
 
        if (memcache_lookup(NULL, SINGLETON_CACHE,
-                       data_blob_string_const("get_mydnsfullname"),
+                       data_blob_string_const_null("get_mydnsfullname"),
                        &tmp)) {
                SMB_ASSERT(tmp.length > 0);
                return (const char *)tmp.data;
@@ -2072,11 +1584,11 @@ const char *get_mydnsfullname(void)
         */
 
        memcache_add(NULL, SINGLETON_CACHE,
-                       data_blob_string_const("get_mydnsfullname"),
-                       data_blob_string_const(res->ai_canonname));
+                       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("get_mydnsfullname"),
+                       data_blob_string_const_null("get_mydnsfullname"),
                        &tmp)) {
                tmp = data_blob_talloc(talloc_tos(), res->ai_canonname,
                                strlen(res->ai_canonname) + 1);
@@ -2163,7 +1675,8 @@ bool is_myname_or_ipaddr(const char *s)
                        return false;
                }
 
-               if (is_zero_addr(&ss) || is_loopback_addr(&ss)) {
+               if (is_zero_addr((struct sockaddr *)&ss) || 
+                       is_loopback_addr((struct sockaddr *)&ss)) {
                        return false;
                }
 
@@ -2174,7 +1687,7 @@ bool is_myname_or_ipaddr(const char *s)
                }
                n = get_interfaces(nics, MAX_INTERFACES);
                for (i=0; i<n; i++) {
-                       if (addr_equal(&nics[i].ip, &ss)) {
+                       if (sockaddr_equal((struct sockaddr *)&nics[i].ip, (struct sockaddr *)&ss)) {
                                TALLOC_FREE(nics);
                                return true;
                        }