udp: using datalen to cap ipv6 udp max gso segments
authorCoco Li <lixiaoyan@google.com>
Thu, 23 Dec 2021 22:24:40 +0000 (22:24 +0000)
committerJakub Kicinski <kuba@kernel.org>
Fri, 24 Dec 2021 03:20:06 +0000 (19:20 -0800)
The max number of UDP gso segments is intended to cap to
UDP_MAX_SEGMENTS, this is checked in udp_send_skb().

skb->len contains network and transport header len here, we should use
only data len instead.

This is the ipv6 counterpart to the below referenced commit,
which missed the ipv6 change

Fixes: 158390e45612 ("udp: using datalen to cap max gso segments")
Signed-off-by: Coco Li <lixiaoyan@google.com>
Reviewed-by: Willem de Bruijn <willemb@google.com>
Link: https://lore.kernel.org/r/20211223222441.2975883-1-lixiaoyan@google.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
net/ipv6/udp.c

index a2caca6ccf114546f9e4ea854ad67208b2f3873e..8cde9efd79198adc1a83260d0ffad21ee3b185ae 100644 (file)
@@ -1204,7 +1204,7 @@ static int udp_v6_send_skb(struct sk_buff *skb, struct flowi6 *fl6,
                        kfree_skb(skb);
                        return -EINVAL;
                }
-               if (skb->len > cork->gso_size * UDP_MAX_SEGMENTS) {
+               if (datalen > cork->gso_size * UDP_MAX_SEGMENTS) {
                        kfree_skb(skb);
                        return -EINVAL;
                }