From b12c1e7cf744a5e28db5a29e89377496893c1ab8 Mon Sep 17 00:00:00 2001 From: Lennart Poettering Date: Sun, 11 Jul 2010 02:23:11 +0200 Subject: [PATCH] socket: pass minimal abstract socket names --- src/manager.c | 2 +- src/socket-util.c | 10 +++------- 2 files changed, 4 insertions(+), 8 deletions(-) diff --git a/src/manager.c b/src/manager.c index d26541d3..6dfef713 100644 --- a/src/manager.c +++ b/src/manager.c @@ -89,7 +89,7 @@ static int manager_setup_notify(Manager *m) { else strncpy(sa.un.sun_path+1, NOTIFY_SOCKET, sizeof(sa.un.sun_path)-1); - if (bind(m->notify_watch.fd, &sa.sa, sizeof(sa)) < 0) { + if (bind(m->notify_watch.fd, &sa.sa, sizeof(sa_family_t) + 1 + strlen(sa.un.sun_path+1)) < 0) { log_error("bind() failed: %m"); return -errno; } diff --git a/src/socket-util.c b/src/socket-util.c index 5463ffbc..e6e3784b 100644 --- a/src/socket-util.c +++ b/src/socket-util.c @@ -101,7 +101,7 @@ int socket_address_parse(SocketAddress *a, const char *s) { a->sockaddr.un.sun_family = AF_UNIX; memcpy(a->sockaddr.un.sun_path+1, s+1, l); - a->size = sizeof(struct sockaddr_un); + a->size = sizeof(sa_family_t) + 1 + l; } else { @@ -198,11 +198,7 @@ int socket_address_verify(const SocketAddress *a) { if (a->size > sizeof(sa_family_t)) { - if (a->sockaddr.un.sun_path[0] == 0) { - /* abstract */ - if (a->size != sizeof(struct sockaddr_un)) - return -EINVAL; - } else { + if (a->sockaddr.un.sun_path[0] != 0) { char *e; /* path */ @@ -437,7 +433,7 @@ bool socket_address_equal(const SocketAddress *a, const SocketAddress *b) { if (strncmp(a->sockaddr.un.sun_path, b->sockaddr.un.sun_path, sizeof(a->sockaddr.un.sun_path)) != 0) return false; } else { - if (memcmp(a->sockaddr.un.sun_path, b->sockaddr.un.sun_path, sizeof(a->sockaddr.un.sun_path)) != 0) + if (memcmp(a->sockaddr.un.sun_path, b->sockaddr.un.sun_path, a->size) != 0) return false; } -- 2.39.5