From: Lennart Poettering Date: Tue, 3 Apr 2012 17:24:12 +0000 (+0200) Subject: fix a couple of AF_UNIX connect() calls X-Git-Tag: v183~458 X-Git-Url: https://err.no/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=3338b9595b69a4a6de43bbd0f49b941eba7f6b0e;p=systemd fix a couple of AF_UNIX connect() calls --- diff --git a/src/bridge.c b/src/bridge.c index 1f7cf3a9..bfb38a8b 100644 --- a/src/bridge.c +++ b/src/bridge.c @@ -160,7 +160,7 @@ int main(int argc, char *argv[]) { sa.un.sun_family = AF_UNIX; strncpy(sa.un.sun_path, "/run/dbus/system_bus_socket", sizeof(sa.un.sun_path)); - if (connect(fd, &sa.sa, offsetof(struct sockaddr_un, sun_path) + 1 + strlen(sa.un.sun_path+1)) < 0) { + if (connect(fd, &sa.sa, offsetof(struct sockaddr_un, sun_path) + strlen(sa.un.sun_path)) < 0) { log_error("Failed to connect: %m"); goto finish; } diff --git a/src/log.c b/src/log.c index 94993676..5b172118 100644 --- a/src/log.c +++ b/src/log.c @@ -151,7 +151,7 @@ static int log_open_syslog(void) { goto fail; } - if (connect(syslog_fd, &sa.sa, sizeof(sa)) < 0) { + if (connect(syslog_fd, &sa.sa, offsetof(struct sockaddr_un, sun_path) + strlen(sa.un.sun_path)) < 0) { close_nointr_nofail(syslog_fd); /* Some legacy syslog systems still use stream @@ -163,7 +163,7 @@ static int log_open_syslog(void) { goto fail; } - if (connect(syslog_fd, &sa.sa, sizeof(sa)) < 0) { + if (connect(syslog_fd, &sa.sa, offsetof(struct sockaddr_un, sun_path) + strlen(sa.un.sun_path)) < 0) { r = -errno; goto fail; }