On Mon, 9 Sept 2024 at 16:27, Mikhail Kshevetskiy <mikhail.kshevets...@iopsys.eu> wrote: > > The wget test seriously broken: > * Uses zero values for ISS and IRS values (see RFC 9293). > * Writes incorrect values to Sequence and Acknowledgment numbers fields > of tcp packets. In the real life such packets will break the tcp stream. > * The test should fail (see above), but due to buggy old tcp implementation > it passes. > > This patch fix all bugs mentioned above, so the test passes with new > and better tcp implementation. > > Signed-off-by: Mikhail Kshevetskiy <mikhail.kshevets...@iopsys.eu> > --- > arch/sandbox/include/asm/eth.h | 4 +++ > test/cmd/wget.c | 45 ++++++++++++++++++++++++---------- > 2 files changed, 36 insertions(+), 13 deletions(-) >
Reviewed-by: Simon Glass <s...@chromium.org> Thanks for tidying this up