Re: [PATCH 1/2] net: nfs: clean up bounds checks in nfs_readlink_reply()

Murtaza <[email protected]>
Newsgroups gmane.comp.boot-loaders.u-boot.general,gmane.comp.boot-loaders.u-boot
Message-ID <CAOqbjtt+UJz0VzYiT=YZA9_h1dHW6155v++2XHL1ueX3JdvD8g@mail.gmail.com>
Hi Jerome,

Thanks for applying the patch! Has it been merged yet? I don't see it in
mainline, thanks!

Best,
Murtaza

On Thu, Jul 23, 2026 at 8:23 AM Jerome Forissier <[email protected]>
wrote:

> On Wed, 22 Jul 2026 23:44:29 +0200, Murtaza Munaim wrote:
> > Commit d6694018eadd ("net: nfs: fix buffer overflow in
> > nfs_readlink_reply()") added bounds checks against sizeof(nfs_path_buff)
> > before both memcpy() calls. This is a cosmetic cleanup of that fix:
> >
> > - introduce a local new_len for the relative-path branch so the sum
> >   pathlen + rlen is computed once and reused for both the bounds check
> >   and the NUL terminator, rather than being open-coded twice;
> > - emit a diagnostic when a symlink target is rejected for exceeding the
> >   buffer, matching the style of other NFS error paths.
> >
> > [...]
>
> Applied to u-boot-net branch for-main, thanks!
>
> [1/2] net: nfs: clean up bounds checks in nfs_readlink_reply()
>       commit: 85d82c52327552da9e9bd103342d970d49ec262c
>
> Best regards,
> --
> Jerome Forissier <[email protected]>
>
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.