summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorChristian Schoenebeck2022-04-29 12:25:29 +0200
committerChristian Schoenebeck2022-05-01 14:07:03 +0200
commit063c75db2e03938b2fadb052c4661adae36e352c (patch)
tree15dc267b4eec4822ffe6ebeb7c5b4c8849936bcb
parent9pfs: fix removing non-existent POSIX ACL xattr on macOS host (diff)
downloadqemu-063c75db2e03938b2fadb052c4661adae36e352c.tar.gz
qemu-063c75db2e03938b2fadb052c4661adae36e352c.tar.xz
qemu-063c75db2e03938b2fadb052c4661adae36e352c.zip
9pfs: fix qemu_mknodat() to always return -1 on error on macOS host
qemu_mknodat() is expected to behave according to its POSIX API, and therefore should always return exactly -1 on any error, and errno should be set for the actual error code. Signed-off-by: Christian Schoenebeck <qemu_oss@crudebyte.com> Reviewed-by: Greg Kurz <groug@kaod.org> Reviewed-by: Akihiko Odaki <akihiko.odaki@gmail.com> Message-Id: <c714b5e1cae225ab7575242c45ee0fe4945eb6ad.1651228001.git.qemu_oss@crudebyte.com>
-rw-r--r--hw/9pfs/9p-util-darwin.c3
1 files changed, 2 insertions, 1 deletions
diff --git a/hw/9pfs/9p-util-darwin.c b/hw/9pfs/9p-util-darwin.c
index bd2dd81548..95146e7354 100644
--- a/hw/9pfs/9p-util-darwin.c
+++ b/hw/9pfs/9p-util-darwin.c
@@ -124,7 +124,8 @@ int qemu_mknodat(int dirfd, const char *filename, mode_t mode, dev_t dev)
}
if (!pthread_fchdir_np) {
error_report_once("pthread_fchdir_np() not available on this version of macOS");
- return -ENOTSUP;
+ errno = ENOTSUP;
+ return -1;
}
if (pthread_fchdir_np(dirfd) < 0) {
return -1;