Re: [PATCH v1 2/9] fs/fuse: add FUSE_OWNER_UID_GID_EXT extension
From: Miklos Szeredi
Date: Thu Aug 29 2024 - 08:30:27 EST
On Thu, 29 Aug 2024 at 14:08, Christian Brauner <brauner@xxxxxxxxxx> wrote:
> Fwiw, that's what the patchset is doing. It's only supported if the
> server sets "default_permissions".
My specific issue is with FUSE_EXT_OWNER_UID_GID, which I think is
unnecessary. Just fill the header with the mapped uid/gid values,
which most servers already use for creating the file with the correct
st_uid/st_gid and not for checking permission. When the mapped values
are unavailable, set the uid/gid in the header -1. Should be better
than sending nonsense values to userspace, no?
Thanks,
Miklos