Re: [PATCH 6/6] net: tftp: sanitize tftp block size, especially for TX

2022-11-28 Thread Tom Rini
On Fri, Oct 14, 2022 at 07:43:42PM +0200, Rasmus Villemoes wrote: > U-Boot does not support IP fragmentation on TX (and unless > CONFIG_IP_DEFRAG is set, neither on RX). So the blocks we send must > fit in a single ethernet packet. > > Currently, if tftpblocksize is set to something like 5000

Re: [PATCH 6/6] net: tftp: sanitize tftp block size, especially for TX

2022-10-16 Thread Ramon Fried
On Fri, Oct 14, 2022 at 8:44 PM Rasmus Villemoes wrote: > > U-Boot does not support IP fragmentation on TX (and unless > CONFIG_IP_DEFRAG is set, neither on RX). So the blocks we send must > fit in a single ethernet packet. > > Currently, if tftpblocksize is set to something like 5000 and I >

[PATCH 6/6] net: tftp: sanitize tftp block size, especially for TX

2022-10-14 Thread Rasmus Villemoes
U-Boot does not support IP fragmentation on TX (and unless CONFIG_IP_DEFRAG is set, neither on RX). So the blocks we send must fit in a single ethernet packet. Currently, if tftpblocksize is set to something like 5000 and I tftpput a large enough file, U-Boot crashes because we overflow