mail archive of the barebox mailing list
 help / color / mirror / Atom feed
From: Ahmad Fatoum <a.fatoum@pengutronix.de>
To: barebox@lists.infradead.org
Subject: [PATCH 00/10] net: fix problems handling trailing bytes
Date: Thu,  4 Apr 2024 20:39:51 +0200	[thread overview]
Message-ID: <20240404184001.1532897-1-a.fatoum@pengutronix.de> (raw)

This started by pinging a Raspberry Pi 3b running barebox sitting behind
a router employing conntrack. The router discarded the ping responses
due to the wrong ICMP checksum and these issues do not pop up normally
because the ping command itself doesn't bother to verify the checksum.

This unearthed issues in two drivers as well as the barebox network
stack itself.

See the commit messages for details.

Ahmad Fatoum (10):
  net: free packets with net_free_packet
  net: ip: don't blindly trust driver supplied frame size
  net: icmp: don't blindly trust driver supplied frame size
  net: icmp: properly set IP TTL and fragement fields
  net: icmp: don't overrun buffer on send
  net: cpsw: report correct frame size to network stack
  net: usb: smsc95xx: don't opencode get/put_aligned_le32
  net: usb: smsc95xx: don't blindly trust hardware size
  net: usb: smsc95xx: fix handling of multiple packets per urb
  net: usb: smsc95xx: disable HW checksumming in driver

 drivers/net/cpsw.c         |  7 +++-
 drivers/net/usb/smsc95xx.c | 68 ++++++++++++++++----------------------
 net/net.c                  | 64 +++++++++++++++++++++++++++++------
 3 files changed, 88 insertions(+), 51 deletions(-)

-- 
2.39.2




             reply	other threads:[~2024-04-04 18:40 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-04-04 18:39 Ahmad Fatoum [this message]
2024-04-04 18:39 ` [PATCH 01/10] net: free packets with net_free_packet Ahmad Fatoum
2024-04-04 18:39 ` [PATCH 02/10] net: ip: don't blindly trust driver supplied frame size Ahmad Fatoum
2024-04-04 18:39 ` [PATCH 03/10] net: icmp: " Ahmad Fatoum
2024-04-04 18:39 ` [PATCH 04/10] net: icmp: properly set IP TTL and fragement fields Ahmad Fatoum
2024-04-04 18:39 ` [PATCH 05/10] net: icmp: don't overrun buffer on send Ahmad Fatoum
2024-04-04 18:39 ` [PATCH 06/10] net: cpsw: report correct frame size to network stack Ahmad Fatoum
2024-04-04 18:39 ` [PATCH 07/10] net: usb: smsc95xx: don't opencode get/put_aligned_le32 Ahmad Fatoum
2024-04-04 18:39 ` [PATCH 08/10] net: usb: smsc95xx: don't blindly trust hardware size Ahmad Fatoum
2024-04-04 18:40 ` [PATCH 09/10] net: usb: smsc95xx: fix handling of multiple packets per urb Ahmad Fatoum
2024-04-05  7:18   ` [PATCH] fixup! " Ahmad Fatoum
2024-04-04 18:40 ` [PATCH 10/10] net: usb: smsc95xx: disable HW checksumming in driver Ahmad Fatoum
2024-04-04 19:49 ` [PATCH 00/10] net: fix problems handling trailing bytes Ahmad Fatoum
2024-04-05  9:57 ` Sascha Hauer

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20240404184001.1532897-1-a.fatoum@pengutronix.de \
    --to=a.fatoum@pengutronix.de \
    --cc=barebox@lists.infradead.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox