Skip to content

Commit 84c4547

Browse files
committed
Simplify socket address length usage
Drop unnecessary of an unused local variable and calculate the size via the already available address union member.
1 parent 6f0863f commit 84c4547

File tree

3 files changed

+6
-39
lines changed

3 files changed

+6
-39
lines changed

src/netlog/netlog-dtls.c

Lines changed: 2 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -55,7 +55,6 @@ int dtls_connect(DTLSManager *m, SocketAddress *address) {
5555
_cleanup_(SSL_freep) SSL *ssl = NULL;
5656
_cleanup_free_ char *pretty = NULL;
5757
const SSL_CIPHER *cipher;
58-
union sockaddr_union sa;
5958
socklen_t salen;
6059
SSL_CTX *ctx;
6160
struct timeval timeout = {
@@ -69,20 +68,10 @@ int dtls_connect(DTLSManager *m, SocketAddress *address) {
6968

7069
switch (address->sockaddr.sa.sa_family) {
7170
case AF_INET:
72-
sa = (union sockaddr_union) {
73-
.in.sin_family = address->sockaddr.sa.sa_family,
74-
.in.sin_port = address->sockaddr.in.sin_port,
75-
.in.sin_addr = address->sockaddr.in.sin_addr,
76-
};
77-
salen = sizeof(sa.in);
71+
salen = sizeof(address->sockaddr.in);
7872
break;
7973
case AF_INET6:
80-
sa = (union sockaddr_union) {
81-
.in6.sin6_family = address->sockaddr.sa.sa_family,
82-
.in6.sin6_port = address->sockaddr.in6.sin6_port,
83-
.in6.sin6_addr = address->sockaddr.in6.sin6_addr,
84-
};
85-
salen = sizeof(sa.in6);
74+
salen = sizeof(address->sockaddr.in6);
8675
break;
8776
default:
8877
return -EAFNOSUPPORT;

src/netlog/netlog-network.c

Lines changed: 2 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -135,7 +135,6 @@ void manager_close_network_socket(Manager *m) {
135135

136136
int manager_network_connect_socket(Manager *m) {
137137
_cleanup_free_ char *pretty = NULL;
138-
union sockaddr_union sa;
139138
socklen_t salen;
140139
int r;
141140

@@ -144,20 +143,10 @@ int manager_network_connect_socket(Manager *m) {
144143

145144
switch (m->address.sockaddr.sa.sa_family) {
146145
case AF_INET:
147-
sa = (union sockaddr_union) {
148-
.in.sin_family = m->address.sockaddr.sa.sa_family,
149-
.in.sin_port = m->address.sockaddr.in.sin_port,
150-
.in.sin_addr = m->address.sockaddr.in.sin_addr,
151-
};
152-
salen = sizeof(sa.in);
146+
salen = sizeof(m->address.sockaddr.in);
153147
break;
154148
case AF_INET6:
155-
sa = (union sockaddr_union) {
156-
.in6.sin6_family = m->address.sockaddr.sa.sa_family,
157-
.in6.sin6_port = m->address.sockaddr.in6.sin6_port,
158-
.in6.sin6_addr = m->address.sockaddr.in6.sin6_addr,
159-
};
160-
salen = sizeof(sa.in6);
149+
salen = sizeof(m->address.sockaddr.in6);
161150
break;
162151
default:
163152
return -EAFNOSUPPORT;

src/netlog/netlog-tls.c

Lines changed: 2 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -163,7 +163,6 @@ int tls_connect(TLSManager *m, SocketAddress *address) {
163163
_cleanup_(SSL_freep) SSL *ssl = NULL;
164164
_cleanup_free_ char *pretty = NULL;
165165
const SSL_CIPHER *cipher;
166-
union sockaddr_union sa;
167166
socklen_t salen;
168167
SSL_CTX *ctx;
169168
int fd, r;
@@ -173,20 +172,10 @@ int tls_connect(TLSManager *m, SocketAddress *address) {
173172

174173
switch (address->sockaddr.sa.sa_family) {
175174
case AF_INET:
176-
sa = (union sockaddr_union) {
177-
.in.sin_family = address->sockaddr.sa.sa_family,
178-
.in.sin_port = address->sockaddr.in.sin_port,
179-
.in.sin_addr = address->sockaddr.in.sin_addr,
180-
};
181-
salen = sizeof(sa.in);
175+
salen = sizeof(address->sockaddr.in);
182176
break;
183177
case AF_INET6:
184-
sa = (union sockaddr_union) {
185-
.in6.sin6_family = address->sockaddr.sa.sa_family,
186-
.in6.sin6_port = address->sockaddr.in6.sin6_port,
187-
.in6.sin6_addr = address->sockaddr.in6.sin6_addr,
188-
};
189-
salen = sizeof(sa.in6);
178+
salen = sizeof(address->sockaddr.in6);
190179
break;
191180
default:
192181
return -EAFNOSUPPORT;

0 commit comments

Comments
 (0)