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

Hi Murtaza,

On 12/08/2026 08:09, Murtaza wrote:
> Hi Jerome,
> 
> Thanks for applying the patch! Has it been merged yet? I don't see it in mainline, thanks!

I haven't sent a pull request to Tom yet but I will do soon.

Thanks!
-- 
Jerome

> 
> Best,
> Murtaza
> 
> On Thu, Jul 23, 2026 at 8:23 AM Jerome Forissier <[email protected] <mailto:[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] <mailto:[email protected]>>
>