mir/sudo
2
0
Fork 0
mirror of https://github.com/sudo-project/sudo.git synced 2025-08-28 12:57:50 +00:00
Code Issues Releases Wiki Activity

union sockaddr_union: pass in sockaddr_union * instead of sockaddr *.

Browse Source 
This eliminates the need for a few casts and is consistent with how
create_listener() is written.
This commit is contained in:
Todd C. Miller 2022-06-03 10:11:37 -06:00
parent 898ca50545
commit 5012c7f978
1 changed files with 10 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
logsrvd/logsrvd.c
View File

@ -1364,7 +1364,7 @@ bad:
* Allocate a connection closure and optionally perform TLS handshake. * Allocate a connection closure and optionally perform TLS handshake.
*/ */
static bool static bool
new_connection(int sock, bool tls, const struct sockaddr *sa, new_connection(int sock, bool tls, const union sockaddr_union *sa_un,
struct sudo_event_base *evbase) struct sudo_event_base *evbase)
{ {
struct connection_closure *closure; struct connection_closure *closure;
@ -1374,14 +1374,12 @@ new_connection(int sock, bool tls, const struct sockaddr *sa,
goto bad; goto bad;
/* store the peer's IP address in the closure object */ /* store the peer's IP address in the closure object */
if (sa->sa_family == AF_INET) { if (sa_un->sa.sa_family == AF_INET) {
struct sockaddr_in *sin = (struct sockaddr_in *)sa; inet_ntop(AF_INET, &sa_un->sin.sin_addr, closure->ipaddr,
inet_ntop(AF_INET, &sin->sin_addr, closure->ipaddr,
sizeof(closure->ipaddr)); sizeof(closure->ipaddr));
#if defined(HAVE_STRUCT_IN6_ADDR) #if defined(HAVE_STRUCT_IN6_ADDR)
} else if (sa->sa_family == AF_INET6) { } else if (sa_un->sa.sa_family == AF_INET6) {
struct sockaddr_in6 *sin6 = (struct sockaddr_in6 *)sa; inet_ntop(AF_INET6, &sa_un->sin6.sin6_addr, closure->ipaddr,
inet_ntop(AF_INET6, &sin6->sin6_addr, closure->ipaddr,
sizeof(closure->ipaddr)); sizeof(closure->ipaddr));
#endif /* HAVE_STRUCT_IN6_ADDR */ #endif /* HAVE_STRUCT_IN6_ADDR */
} else { } else {
@ -1500,12 +1498,12 @@ listener_cb(int fd, int what, void *v)
{ {
struct listener *l = v; struct listener *l = v;
struct sudo_event_base *evbase = sudo_ev_get_base(l->ev); struct sudo_event_base *evbase = sudo_ev_get_base(l->ev);
union sockaddr_union s_un; union sockaddr_union sa_un;
socklen_t salen = sizeof(s_un); socklen_t salen = sizeof(sa_un);
int sock; int sock;
debug_decl(listener_cb, SUDO_DEBUG_UTIL); debug_decl(listener_cb, SUDO_DEBUG_UTIL);
sock = accept(fd, &s_un.sa, &salen); sock = accept(fd, &sa_un.sa, &salen);
if (sock != -1) { if (sock != -1) {
if (logsrvd_conf_server_tcp_keepalive()) { if (logsrvd_conf_server_tcp_keepalive()) {
int keepalive = 1; int keepalive = 1;
@ -1514,7 +1512,7 @@ listener_cb(int fd, int what, void *v)
sudo_warn("SO_KEEPALIVE"); sudo_warn("SO_KEEPALIVE");
} }
} }
if (!new_connection(sock, l->tls, &s_un.sa, evbase)) { if (!new_connection(sock, l->tls, &sa_un, evbase)) {
/* TODO: pause accepting on ENOMEM */ /* TODO: pause accepting on ENOMEM */
sudo_debug_printf(SUDO_DEBUG_ERROR|SUDO_DEBUG_LINENO, sudo_debug_printf(SUDO_DEBUG_ERROR|SUDO_DEBUG_LINENO,
"unable to start new connection"); "unable to start new connection");