From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 2613AA0A02; Thu, 25 Mar 2021 11:48:33 +0100 (CET) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id A00D64067B; Thu, 25 Mar 2021 11:48:32 +0100 (CET) Received: from qq.com (out162-62-57-87.mail.qq.com [162.62.57.87]) by mails.dpdk.org (Postfix) with ESMTP id 8DF7740147 for ; Thu, 25 Mar 2021 11:48:30 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=qq.com; s=s201512; t=1616669308; bh=2ee4kJTu4Kin8FfYNLmpVJvV2oJVYLewarTBSKKqzwU=; h=From:To:Cc:Subject:Date:In-Reply-To:References; b=ojwXlKa0RR5qMGgFyiqIukQ/NAhRAO6nBdk/f8cqf+ry6qUIg9bvZdNivgCWB7XqX hHl2iEGDPxkhGb6ISSUfDaQDhIy3B+0uDqR5Ncr7cPjD0bQfh/VaIOeAogRidq9YtD eYDT6hPJES3psASV8c9cT2hLoWYya0VbY0DfCba8= Received: from localhost.localdomain ([2409:8a20:202e:ae80:1976:bbc3:2885:f49c]) by newxmesmtplogicsvrsza5.qq.com (NewEsmtp) with SMTP id C1B3645D; Thu, 25 Mar 2021 18:48:27 +0800 X-QQ-mid: xmsmtpt1616669307tz2j49iya Message-ID: X-QQ-XMAILINFO: MqDJPKMI4IpJ8auDBzcuDqOHn9Spyh4zi2SBDF6oGtunSSDYcThgYA05p3T1a+ ddAJCLGMoGgBmTbxkinMpu3wE71qcAWn8MXC92LVapSO56kvrRn5YZbuaiUfZSwyboIrEfNGHBXC Djy/ll0h17g1LSaR9KJNdbaotM8sxX3vuIuqOcdlf8M6v+cFkZtKwpT9OvaSNemxn9ebUq8wVCmd jLiwDGDgtpifSV6Lmr4KUWXWzMJx3Dfyo8ZPJEJRrE8QiUlJEue+Y9VVBnvpcHbcD7Xn0sVrgB2W t08NzuD3TN09gnhcI3XWB1oJPb+tPPueCf9J1OtqEveAahW1iuhiN2rnMSzDL6KL1ctQVTwMovxC dNPE0z2jyQLG5/CE+EEea3ccYVOtTz8vXZSC3CdFA/MhUd4XQNlfk9tiCTgkQDiIl+TrSqapNOom 6BixFf/TSR8fO9J7JGy3FZ9mAKY1ugzobSpN+iK2zLuw8kaFF5/lr3vxJipmUZiLR6Yga4tMNYWx LL7HcP0IMF+obifrc5yuxOz1sMSVPSwvQooOFk9NPTL10W36RHoamkD/TFaw5roWKyBjUPyHjbYx VE3UsnGSqeQ+Lvi4e38z9qc0kNdYPyJLlgiAmYow3cj8Wk7zCsgSyRgVk1fqYQr0T9SRWbZnD5Tr 0FQJilatCLtow6f1TaPeVRZZsDT5nMwTP9fYrstaPopyt7FzyHtNZlP7Q5H5La6gCWO5h7m4js/5 t1XVoG1fwT/2j46UvPQtoNyoJ1NtPuGv02kcaw1XAwHtCP32i5FbV3i+Ez9iZwDCdm3tythzVKcD 1BWIRhS6D7rIsWnvcFMITA8QtxxRPN/wF6LwWqslMZZ8KxYrsEl+j2FgVGu1LHWM44Rsp/MdRCMf +z/6pCSjCSoEmFyLII5gv3AQG2TyXS0E3ZISp7AwtSPwHxGYhwqYkiJdYyqDmkvw== From: 583493798@qq.com To: dev@dpdk.org Cc: thomas@monjalon.net, Pu Xu <583493798@qq.com>, Konstantin Ananyev Date: Thu, 25 Mar 2021 18:48:25 +0800 X-OQ-MSGID: <20210325104825.94563-1-583493798@qq.com> X-Mailer: git-send-email 2.17.0 In-Reply-To: <20200501161603.33907-1-583493798@qq.com> References: <20200501161603.33907-1-583493798@qq.com> Subject: [dpdk-dev] [PATCH v2] When fragmenting ipv4 packet, the data offset should be calculated through the ihl field in ip header rather than using sizeof(struct rte_ipv4_hdr). X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" From: Pu Xu <583493798@qq.com> Signed-off-by: Pu Xu <583493798@qq.com> --- lib/librte_ip_frag/rte_ipv4_fragmentation.c | 34 +++++++++++++-------- 1 file changed, 21 insertions(+), 13 deletions(-) diff --git a/lib/librte_ip_frag/rte_ipv4_fragmentation.c b/lib/librte_ip_frag/rte_ipv4_fragmentation.c index e9de335ae..2e7739d02 100644 --- a/lib/librte_ip_frag/rte_ipv4_fragmentation.c +++ b/lib/librte_ip_frag/rte_ipv4_fragmentation.c @@ -23,10 +23,10 @@ #define IPV4_HDR_FO_ALIGN (1 << RTE_IPV4_HDR_FO_SHIFT) static inline void __fill_ipv4hdr_frag(struct rte_ipv4_hdr *dst, - const struct rte_ipv4_hdr *src, uint16_t len, uint16_t fofs, - uint16_t dofs, uint32_t mf) + const struct rte_ipv4_hdr *src, uint16_t header_len, + uint16_t len, uint16_t fofs, uint16_t dofs, uint32_t mf) { - rte_memcpy(dst, src, sizeof(*dst)); + rte_memcpy(dst, src, header_len); fofs = (uint16_t)(fofs + (dofs >> RTE_IPV4_HDR_FO_SHIFT)); fofs = (uint16_t)(fofs | mf << RTE_IPV4_HDR_MF_SHIFT); dst->fragment_offset = rte_cpu_to_be_16(fofs); @@ -74,7 +74,7 @@ rte_ipv4_fragment_packet(struct rte_mbuf *pkt_in, struct rte_ipv4_hdr *in_hdr; uint32_t out_pkt_pos, in_seg_data_pos; uint32_t more_in_segs; - uint16_t fragment_offset, flag_offset, frag_size; + uint16_t fragment_offset, flag_offset, frag_size, header_len; uint16_t frag_bytes_remaining; /* @@ -86,14 +86,22 @@ rte_ipv4_fragment_packet(struct rte_mbuf *pkt_in, unlikely(mtu_size < RTE_ETHER_MIN_MTU)) return -EINVAL; + in_hdr = rte_pktmbuf_mtod(pkt_in, struct rte_ipv4_hdr *); + header_len = (in_hdr->version_ihl & RTE_IPV4_HDR_IHL_MASK) * + RTE_IPV4_IHL_MULTIPLIER; + + /* Check IP header length */ + if (unlikely(pkt_in->data_len < header_len) || + unlikely(mtu_size < header_len)) + return -EINVAL; + /* * Ensure the IP payload length of all fragments is aligned to a * multiple of 8 bytes as per RFC791 section 2.3. */ - frag_size = RTE_ALIGN_FLOOR((mtu_size - sizeof(struct rte_ipv4_hdr)), + frag_size = RTE_ALIGN_FLOOR((mtu_size - header_len), IPV4_HDR_FO_ALIGN); - in_hdr = rte_pktmbuf_mtod(pkt_in, struct rte_ipv4_hdr *); flag_offset = rte_cpu_to_be_16(in_hdr->fragment_offset); /* If Don't Fragment flag is set */ @@ -102,11 +110,11 @@ rte_ipv4_fragment_packet(struct rte_mbuf *pkt_in, /* Check that pkts_out is big enough to hold all fragments */ if (unlikely(frag_size * nb_pkts_out < - (uint16_t)(pkt_in->pkt_len - sizeof(struct rte_ipv4_hdr)))) + (uint16_t)(pkt_in->pkt_len - header_len))) return -EINVAL; in_seg = pkt_in; - in_seg_data_pos = sizeof(struct rte_ipv4_hdr); + in_seg_data_pos = header_len; out_pkt_pos = 0; fragment_offset = 0; @@ -124,8 +132,8 @@ rte_ipv4_fragment_packet(struct rte_mbuf *pkt_in, } /* Reserve space for the IP header that will be built later */ - out_pkt->data_len = sizeof(struct rte_ipv4_hdr); - out_pkt->pkt_len = sizeof(struct rte_ipv4_hdr); + out_pkt->data_len = header_len; + out_pkt->pkt_len = header_len; frag_bytes_remaining = frag_size; out_seg_prev = out_pkt; @@ -176,14 +184,14 @@ rte_ipv4_fragment_packet(struct rte_mbuf *pkt_in, out_hdr = rte_pktmbuf_mtod(out_pkt, struct rte_ipv4_hdr *); - __fill_ipv4hdr_frag(out_hdr, in_hdr, + __fill_ipv4hdr_frag(out_hdr, in_hdr, header_len, (uint16_t)out_pkt->pkt_len, flag_offset, fragment_offset, more_in_segs); fragment_offset = (uint16_t)(fragment_offset + - out_pkt->pkt_len - sizeof(struct rte_ipv4_hdr)); + out_pkt->pkt_len - header_len); - out_pkt->l3_len = sizeof(struct rte_ipv4_hdr); + out_pkt->l3_len = header_len; /* Write the fragment to the output list */ pkts_out[out_pkt_pos] = out_pkt; -- 2.17.0