test/reorder: fix out of bound access
authorFerruh Yigit <ferruh.yigit@intel.com>
Tue, 13 Nov 2018 23:31:37 +0000 (23:31 +0000)
committerThomas Monjalon <thomas@monjalon.net>
Wed, 14 Nov 2018 03:55:43 +0000 (04:55 +0100)
The value of array index 'i' is out of bound because of the previous
loop it has been used.

Assuming intention is using '0' since the check before free is robufs[0]
check, fixing according.

Fixes: ecd867faa860 ("test/reorder: fix freeing mbuf twice")
Cc: stable@dpdk.org
Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>
test/test/test_reorder.c

index ccee4d0..58fa9c7 100644 (file)
@@ -269,7 +269,7 @@ test_reorder_drain(void)
                goto exit;
        }
        if (robufs[0] != NULL)
-               rte_pktmbuf_free(robufs[i]);
+               rte_pktmbuf_free(robufs[0]);
 
        /* Insert more packets
         * RB[] = {NULL, NULL, NULL, NULL}