vmxnet3: remove asserts that confuse coverity
authorStephen Hemminger <stephen@networkplumber.org>
Wed, 30 Mar 2016 21:38:25 +0000 (14:38 -0700)
committerThomas Monjalon <thomas.monjalon@6wind.com>
Thu, 31 Mar 2016 19:18:55 +0000 (21:18 +0200)
These asserts are only for debugging and never fired during
any testing, but they confuse coverity's null tracking.

Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
Acked-by: Yong Wang <yongwang@vmware.com>
drivers/net/vmxnet3/vmxnet3_rxtx.c

index ebf883f..4ac0456 100644 (file)
@@ -710,7 +710,6 @@ vmxnet3_recv_pkts(void *rx_queue, struct rte_mbuf **rx_pkts, uint16_t nb_pkts)
                 * the last mbuf of the current packet.
                 */
                if (rcd->sop) {
-                       VMXNET3_ASSERT(rxq->start_seg != NULL);
                        VMXNET3_ASSERT(rxd->btype == VMXNET3_RXD_BTYPE_HEAD);
 
                        if (unlikely(rcd->len == 0)) {
@@ -729,7 +728,6 @@ vmxnet3_recv_pkts(void *rx_queue, struct rte_mbuf **rx_pkts, uint16_t nb_pkts)
                        struct rte_mbuf *start = rxq->start_seg;
 
                        VMXNET3_ASSERT(rxd->btype == VMXNET3_RXD_BTYPE_BODY);
-                       VMXNET3_ASSERT(start != NULL);
 
                        start->pkt_len += rxm->data_len;
                        start->nb_segs++;