From: Ferruh Yigit Date: Tue, 16 Apr 2019 15:51:26 +0000 (+0100) Subject: net: forbid VLAN insert in shared mbuf X-Git-Url: http://git.droids-corp.org/?a=commitdiff_plain;h=15a74163b12ed9b8b980b1576bdd8de16d60612b;p=dpdk.git net: forbid VLAN insert in shared mbuf The vlan_insert() is buggy when it tries to handle the shared mbufs, instead don't support inserting VLAN tag into shared mbufs and return an error for that case. Signed-off-by: Ferruh Yigit Acked-by: Olivier Matz --- diff --git a/lib/librte_net/rte_ether.h b/lib/librte_net/rte_ether.h index 7be9b4890a..1868cb7077 100644 --- a/lib/librte_net/rte_ether.h +++ b/lib/librte_net/rte_ether.h @@ -393,15 +393,8 @@ static inline int rte_vlan_insert(struct rte_mbuf **m) struct rte_vlan_hdr *vh; /* Can't insert header if mbuf is shared */ - if (rte_mbuf_refcnt_read(*m) > 1) { - struct rte_mbuf *copy; - - copy = rte_pktmbuf_clone(*m, (*m)->pool); - if (unlikely(copy == NULL)) - return -ENOMEM; - rte_pktmbuf_free(*m); - *m = copy; - } + if (!RTE_MBUF_DIRECT(*m) || rte_mbuf_refcnt_read(*m) > 1) + return -EINVAL; oh = rte_pktmbuf_mtod(*m, struct rte_ether_hdr *); nh = (struct rte_ether_hdr *)