Skip to content

Commit 5b8304b

Browse files
committed
Drop Manager member socklen
The member socklen is used only locally in a single function, and always assigned before usage.
1 parent 04469d1 commit 5b8304b

File tree

2 files changed

+1
-3
lines changed

2 files changed

+1
-3
lines changed

src/netlog/netlog-manager.c

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -506,15 +506,14 @@ int manager_resolve_handler(sd_resolve_query *q, int ret, const struct addrinfo
506506
continue;
507507
}
508508

509-
m->socklen = ai->ai_addrlen;
510509
memcpy(&m->address.sockaddr, (const union sockaddr_union*) ai->ai_addr, ai->ai_addrlen);
511510

512511
if (ai->ai_addr->sa_family == AF_INET6)
513512
m->address.sockaddr.in6.sin6_port = htobe16((uint16_t) m->port);
514513
else
515514
m->address.sockaddr.in.sin_port = htobe16((uint16_t) m->port);
516515

517-
sockaddr_pretty(&m->address.sockaddr.sa, m->socklen, true, true, &pretty);
516+
sockaddr_pretty(&m->address.sockaddr.sa, ai->ai_addrlen, true, true, &pretty);
518517

519518
log_debug("Resolved address %s for %s.", pretty, m->server_name);
520519

src/netlog/netlog-manager.h

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,6 @@ struct Manager {
5454

5555
/* Multicast UDP address */
5656
SocketAddress address;
57-
socklen_t socklen;
5857
uint32_t port;
5958

6059
char *server_name;

0 commit comments

Comments
 (0)