Re: [PATCH v1 2/9] fs/fuse: add FUSE_OWNER_UID_GID_EXT extension

[Miklos Szeredi]

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