9p: proxy: Fix size passed to `connect`
authorKeno Fischer <keno@juliacomputing.com>
Thu, 7 Jun 2018 10:17:21 +0000 (12:17 +0200)
committerGreg Kurz <groug@kaod.org>
Thu, 7 Jun 2018 10:17:21 +0000 (12:17 +0200)
The size to pass to the `connect` call is the size of the entire
`struct sockaddr_un`. Passing anything shorter than this causes errors
on darwin.

Signed-off-by: Keno Fischer <keno@juliacomputing.com>
Signed-off-by: Greg Kurz <groug@kaod.org>
hw/9pfs/9p-proxy.c

index e2e03292de2cf8133d39a061bd379fe7bf85def9..47a94e088daa2a3782258ea45d5935beaa5dc49e 100644 (file)
@@ -1088,7 +1088,7 @@ static int proxy_ioc_getversion(FsContext *fs_ctx, V9fsPath *path,
 
 static int connect_namedsocket(const char *path, Error **errp)
 {
-    int sockfd, size;
+    int sockfd;
     struct sockaddr_un helper;
 
     if (strlen(path) >= sizeof(helper.sun_path)) {
@@ -1102,8 +1102,7 @@ static int connect_namedsocket(const char *path, Error **errp)
     }
     strcpy(helper.sun_path, path);
     helper.sun_family = AF_UNIX;
-    size = strlen(helper.sun_path) + sizeof(helper.sun_family);
-    if (connect(sockfd, (struct sockaddr *)&helper, size) < 0) {
+    if (connect(sockfd, (struct sockaddr *)&helper, sizeof(helper)) < 0) {
         error_setg_errno(errp, errno, "failed to connect to '%s'", path);
         close(sockfd);
         return -1;