On Mon, 2022-12-12 at 06:19 -0800, Heinrich Schuchardt wrote:
«Внимание! Данное письмо от внешнего адресата!»
On 12/7/22 06:52, Vyacheslav Mitrofanov V wrote:
On Wed, 2022-12-07 at 09:42 -0500, Tom Rini wrote:
On Wed, Dec 07, 2022 at 03:29:37PM +0100, Heinrich Schuchardt wrote:
Don't fall through to handling an IPv6 header as IPv4.
Fixes: ffdbf3bad5f3 ("net: ipv6: Incorporate IPv6 support into u- boot net subsystem") Addresses-Coverity-ID: 430975 ("Missing break in switch") Signed-off-by: Heinrich Schuchardt < heinrich.schuchardt@canonical.com>
Do we have a unit trest of IPv6? I guess we should at least do a IPv6 ping.
There are a few tests, yes, but more would be good.
Hello!
I have tested IPv6 (ping6 and tftpboot) with my own python scripts before sending to pathwork. I'll do some test conversion to use them with u-boot or add new tests for that soon! Thanks!
Hello Vycheslav,
would you agree with that break being added? I didn't see a review response.
Best regards
Heinrich
Hello, Heinrich. I'm sorry for the long response, your mail got to spam folder.
Yes I do agree with you that missing break has to be there. I have already sent reviewed sign to your patch.
Many thanks for your work!
P.S. As I promised I'll add soon new tests but it is still in process. Very heavy schedule by the end of the year...