Re: [PATCH 0/14] New uid & gid mount option parsing helpers
Christian Brauner <[email protected]>
| Newsgroups | org.kernel.vger.autofs,dev.linux.lists.ntfs3,org.kernel.vger.linux-cifs,org.kernel.vger.linux-efi,org.kernel.vger.linux-ext4,org.kernel.vger.linux-fsdevel,org.kernel.vger.linux-trace-kernel,org.kvack.linux-mm |
|---|---|
| Message-ID | <20240628-diametral-median-bc0de7b68148@brauner> |
On Thu, Jun 27, 2024 at 07:24:59PM GMT, Eric Sandeen wrote: > Multiple filesystems take uid and gid as options, and the code to > create the ID from an integer and validate it is standard boilerplate > that can be moved into common helper functions, so do that for > consistency and less cut&paste. > > This also helps avoid the buggy pattern noted by Seth Jenkins at > https://lore.kernel.org/lkml/CALxfFW4BXhEwxR0Q5LSkg-8Vb4r2MONKCcUCVioehXQKr35eHg@mail.gmail.com/ > because uid/gid parsing will fail before any assignment in most > filesystems. > > Net effect is a bit of code removal, as well. Thanks, this all looks good to me. I'll have one comment about the fuse patch.