----- Original Message ----- > From: "Philippe Mathieu-Daudé" <[email protected]> > To: [email protected] > Cc: "Jason Wang" <[email protected]>, "Stefano Garzarella" > <[email protected]>, "Thomas Huth" <[email protected]>, > "Miroslav Rezanina" <[email protected]>, "Dmitry Fleytman" > <[email protected]>, "Paolo Bonzini" > <[email protected]>, "Laurent Vivier" <[email protected]>, "Philippe > Mathieu-Daudé" <[email protected]> > Sent: Wednesday, March 10, 2021 7:31:16 PM > Subject: [PATCH v6 0/7] net/eth: Fix stack-buffer-overflow in > _eth_get_rss_ex_dst_addr() > > I had a look at the patch from Miroslav trying to silence a > compiler warning which in fact is a nasty bug. Here is a fix. > https://www.mail-archive.com/[email protected]/msg772735.html >
Reviewed-by: Miroslav Rezanina <[email protected]> > Since v5: > - addressed Stefano's review comments: > - add now patch fixing in6_address offset > > Since v4: > - reworked again, tested it with Fedora Raw Hide > > Philippe Mathieu-Daudé (7): > net/eth: Use correct in6_address offset in _eth_get_rss_ex_dst_addr() > net/eth: Simplify _eth_get_rss_ex_dst_addr() > net/eth: Better describe _eth_get_rss_ex_dst_addr's offset argument > net/eth: Check size earlier in _eth_get_rss_ex_dst_addr() > net/eth: Check iovec has enough data earlier > net/eth: Read ip6_ext_hdr_routing buffer before accessing it > net/eth: Add an assert() and invert if() statement to simplify code > > net/eth.c | 46 ++++++++++++++--------------- > tests/qtest/fuzz-e1000e-test.c | 53 ++++++++++++++++++++++++++++++++++ > MAINTAINERS | 1 + > tests/qtest/meson.build | 1 + > 4 files changed, 78 insertions(+), 23 deletions(-) > create mode 100644 tests/qtest/fuzz-e1000e-test.c > > -- > 2.26.2 > > > -- Miroslav Rezanina Software Engineer - Virtualization Team Maintainer
