diff options
author | Max Reitz <mreitz@redhat.com> | 2021-06-25 16:23:12 +0200 |
---|---|---|
committer | Kevin Wolf <kwolf@redhat.com> | 2021-07-09 12:26:05 +0200 |
commit | 2c7dd057aa7bd7a875e9b1a53975c220d6380bc4 (patch) | |
tree | 38e92d88bf97d121a165c3dd090007389fffe479 /block | |
parent | c2615bdfbd6d5a1a48438102f8ab328fe30b8e27 (diff) | |
download | qemu-2c7dd057aa7bd7a875e9b1a53975c220d6380bc4.zip qemu-2c7dd057aa7bd7a875e9b1a53975c220d6380bc4.tar.gz qemu-2c7dd057aa7bd7a875e9b1a53975c220d6380bc4.tar.bz2 |
export/fuse: Pass default_permissions for mount
We do not do any permission checks in fuse_open(), so let the kernel do
them. We already let fuse_getattr() report the proper UNIX permissions,
so this should work the way we want.
This causes a change in 308's reference output, because now opening a
non-writable export with O_RDWR fails already, instead of only actually
attempting to write to it. (That is an improvement.)
Signed-off-by: Max Reitz <mreitz@redhat.com>
Message-Id: <20210625142317.271673-2-mreitz@redhat.com>
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
Diffstat (limited to 'block')
-rw-r--r-- | block/export/fuse.c | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/block/export/fuse.c b/block/export/fuse.c index 38f74c9..d0b88e8 100644 --- a/block/export/fuse.c +++ b/block/export/fuse.c @@ -153,8 +153,12 @@ static int setup_fuse_export(FuseExport *exp, const char *mountpoint, struct fuse_args fuse_args; int ret; - /* Needs to match what fuse_init() sets. Only max_read must be supplied. */ - mount_opts = g_strdup_printf("max_read=%zu", FUSE_MAX_BOUNCE_BYTES); + /* + * max_read needs to match what fuse_init() sets. + * max_write need not be supplied. + */ + mount_opts = g_strdup_printf("max_read=%zu,default_permissions", + FUSE_MAX_BOUNCE_BYTES); fuse_argv[0] = ""; /* Dummy program name */ fuse_argv[1] = "-o"; |