Skip to content

Commit

Permalink
Merge pull request #4283 from BtbN/fix_unix_fuzzy
Browse files Browse the repository at this point in the history
[Fix] Allow fuzzy worker to listen on AF_UNIX sockets successfully
  • Loading branch information
vstakhov committed Sep 26, 2022
2 parents 60cd969 + e648f72 commit 12430a5
Show file tree
Hide file tree
Showing 2 changed files with 24 additions and 6 deletions.
3 changes: 3 additions & 0 deletions src/fuzzy_storage.c
Expand Up @@ -363,6 +363,9 @@ rspamd_fuzzy_check_write (struct fuzzy_session *session)
}

if (session->ctx->update_ips != NULL && session->addr) {
if (rspamd_inet_address_get_af (session->addr) == AF_UNIX) {
return TRUE;
}
if (rspamd_match_radix_map_addr (session->ctx->update_ips,
session->addr) == NULL) {
return FALSE;
Expand Down
27 changes: 21 additions & 6 deletions src/libutil/addr.c
Expand Up @@ -1060,6 +1060,19 @@ rspamd_inet_address_connect (const rspamd_inet_addr_t *addr, gint type,

if (addr->af == AF_UNIX) {
sa = (const struct sockaddr *)&addr->u.un->addr;

if (type == (int)SOCK_DGRAM) {
struct sockaddr ca;

memset (&ca, 0, sizeof(ca));
ca.sa_family = AF_UNIX;

r = bind (fd, &ca, sizeof (sa_family_t));
if (r == -1) {
msg_info ("unix socket client autobind failed: %s, '%s'",
addr->u.un->addr.sun_path, strerror (errno));
}
}
}
else {
sa = &addr->u.in.addr.sa;
Expand Down Expand Up @@ -1667,7 +1680,8 @@ rspamd_inet_address_from_sa (const struct sockaddr *sa, socklen_t slen)
rspamd_inet_addr_t *addr;

g_assert (sa != NULL);
g_assert (slen >= sizeof (struct sockaddr));
/* Address of an AF_UNIX socket can be tiny */
g_assert (slen >= sizeof (sa_family_t) + 1);

addr = rspamd_inet_addr_create (sa->sa_family, NULL);

Expand All @@ -1676,12 +1690,13 @@ rspamd_inet_address_from_sa (const struct sockaddr *sa, socklen_t slen)
const struct sockaddr_un *un = (const struct sockaddr_un *)sa;

g_assert (slen >= SUN_LEN (un));
g_assert (slen <= sizeof (addr->u.un->addr));

rspamd_strlcpy (addr->u.un->addr.sun_path, un->sun_path,
sizeof (addr->u.un->addr.sun_path));
#if defined(FREEBSD) || defined(__APPLE__)
addr->u.un->addr.sun_len = un->sun_len;
#endif
/* sun_path can legally contain intermittent NULL bytes */
memcpy (&addr->u.un->addr, un, slen);

/* length of AF_UNIX addresses is variable */
addr->slen = slen;
}
else if (sa->sa_family == AF_INET) {
g_assert (slen >= sizeof (struct sockaddr_in));
Expand Down

0 comments on commit 12430a5

Please sign in to comment.