Re: [PATCH v2 17/35] sanitize handling of long-term internal mounts

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, 23 Jun 2025 at 10:03, Al Viro <viro@xxxxxxxxxxxxxxxxxx> wrote:
>
> I don't know...  7 callers with explicit strlen():

Yeah. Most of them clearly just want the string length.

But there were clearly a couple that really didn't have a
NUL-termination thing. And one - that nfs namespace thing - that do
have a string, but get the string length differently.

> > Or maybe even go further and some helper to doi that
> > "fs_context_for_mount()" _with_ a list of param's to be added?
>
> Vararg, presumably?

Or just pass in a descriptor struct / array that can be NULL?

But this really is not a huge deal. I was just looking at your series,
and most things cleaned things up, and this one just grated on me a
bit.

           Linus




[Index of Archives]     [Linux Ext4 Filesystem]     [Union Filesystem]     [Filesystem Testing]     [Ceph Users]     [Ecryptfs]     [NTFS 3]     [AutoFS]     [Kernel Newbies]     [Share Photos]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux Cachefs]     [Reiser Filesystem]     [Linux RAID]     [NTFS 3]     [Samba]     [Device Mapper]     [CEPH Development]

  Powered by Linux