Commit 515ac8d2 authored by Poul-Henning Kamp's avatar Poul-Henning Kamp

More sockaddr->sockaddr_storage cleanup



git-svn-id: http://www.varnish-cache.org/svn/trunk/varnish-cache@5397 d4fa192b-c00b-0410-8231-f00ffab90ce4
parent ee034039
...@@ -156,7 +156,7 @@ copy_sockaddr(struct sockaddr_storage **sa, socklen_t *len, ...@@ -156,7 +156,7 @@ copy_sockaddr(struct sockaddr_storage **sa, socklen_t *len,
{ {
assert(*src > 0); assert(*src > 0);
*sa = malloc(*src); /* XXX: malloc(sizeof **sa) ? */ *sa = malloc(*src);
XXXAN(*sa); XXXAN(*sa);
memcpy(*sa, src + 1, *src); memcpy(*sa, src + 1, *src);
*len = *src; *len = *src;
......
...@@ -30,7 +30,6 @@ ...@@ -30,7 +30,6 @@
*/ */
struct cli; struct cli;
struct sockaddr;
extern pid_t mgt_pid; extern pid_t mgt_pid;
#define ASSERT_MGT() do { assert(getpid() == mgt_pid);} while (0) #define ASSERT_MGT() do { assert(getpid() == mgt_pid);} while (0)
......
...@@ -62,10 +62,7 @@ struct vss_addr { ...@@ -62,10 +62,7 @@ struct vss_addr {
int va_socktype; int va_socktype;
int va_protocol; int va_protocol;
socklen_t va_addrlen; socklen_t va_addrlen;
union { struct sockaddr_storage va_addr;
struct sockaddr_storage _storage;
struct sockaddr sa;
} va_addr;
}; };
/*lint -esym(754, _storage) not ref */ /*lint -esym(754, _storage) not ref */
...@@ -224,7 +221,7 @@ VSS_bind(const struct vss_addr *va) ...@@ -224,7 +221,7 @@ VSS_bind(const struct vss_addr *va)
return (-1); return (-1);
} }
#endif #endif
if (bind(sd, &va->va_addr.sa, va->va_addrlen) != 0) { if (bind(sd, (const void*)&va->va_addr, va->va_addrlen) != 0) {
perror("bind()"); perror("bind()");
(void)close(sd); (void)close(sd);
return (-1); return (-1);
...@@ -272,7 +269,7 @@ VSS_connect(const struct vss_addr *va, int nonblock) ...@@ -272,7 +269,7 @@ VSS_connect(const struct vss_addr *va, int nonblock)
} }
if (nonblock) if (nonblock)
(void)TCP_nonblocking(sd); (void)TCP_nonblocking(sd);
i = connect(sd, &va->va_addr.sa, va->va_addrlen); i = connect(sd, (const void *)&va->va_addr, va->va_addrlen);
if (i == 0 || (nonblock && errno == EINPROGRESS)) if (i == 0 || (nonblock && errno == EINPROGRESS))
return (sd); return (sd);
perror("connect()"); perror("connect()");
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment