gso: Only allow GSO_PARTIAL if we can checksum the inner protocol
authorAlexander Duyck <aduyck@mirantis.com>
Mon, 2 May 2016 16:38:18 +0000 (09:38 -0700)
committerDavid S. Miller <davem@davemloft.net>
Wed, 4 May 2016 17:32:27 +0000 (13:32 -0400)
This patch addresses a possible issue that can occur if we get into any odd
corner cases where we support TSO for a given protocol but not the checksum
or scatter-gather offload.  There are few drivers floating around that
setup their tunnels this way and by enforcing the checksum piece we can
avoid mangling any frames.

Signed-off-by: Alexander Duyck <aduyck@mirantis.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/core/skbuff.c

index b8dd2d2e22560f30b3088b8d157a2e2f94fffa7a..5586be93632fb8ef1d5629d2a0f5faa27290ba74 100644 (file)
@@ -3080,8 +3080,7 @@ struct sk_buff *skb_segment(struct sk_buff *head_skb,
        unsigned int headroom;
        unsigned int len = head_skb->len;
        __be16 proto;
-       bool csum;
-       int sg = !!(features & NETIF_F_SG);
+       bool csum, sg;
        int nfrags = skb_shinfo(head_skb)->nr_frags;
        int err = -ENOMEM;
        int i = 0;
@@ -3093,13 +3092,14 @@ struct sk_buff *skb_segment(struct sk_buff *head_skb,
        if (unlikely(!proto))
                return ERR_PTR(-EINVAL);
 
+       sg = !!(features & NETIF_F_SG);
        csum = !!can_checksum_protocol(features, proto);
 
        /* GSO partial only requires that we trim off any excess that
         * doesn't fit into an MSS sized block, so take care of that
         * now.
         */
-       if (features & NETIF_F_GSO_PARTIAL) {
+       if (sg && csum && (features & NETIF_F_GSO_PARTIAL)) {
                partial_segs = len / mss;
                if (partial_segs > 1)
                        mss *= partial_segs;