[PATCH v2 1/6] fs/readdir: Fix filldir() and filldir64() use of user_access_begin()

Linus Torvalds torvalds at linux-foundation.org
Thu Jan 23 05:24:36 AEDT 2020


[ Sorry about HTML crud, on mobile right now ]

On Wed, Jan 22, 2020, 09:52 Christophe Leroy <christophe.leroy at c-s.fr>
wrote:

>
> -       struct linux_dirent __user * dirent;
> +       struct linux_dirent __user * dirent, *dirent0;


Patch looks better, but those names are horrid.

Please just use "prev" for the previous dirent, and leave the current one
as "dirent".

I think that's going to be a lot more legible. But no numbering.

      Linus
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.ozlabs.org/pipermail/linuxppc-dev/attachments/20200122/238a5219/attachment.htm>


More information about the Linuxppc-dev mailing list