From b0f0f8c8dd9d694bfc13652f89b8b577e9c1c708 Mon Sep 17 00:00:00 2001 From: Mauro Sardara Date: Mon, 14 Mar 2022 20:08:22 +0000 Subject: memif: fix memif_process_desc indexing The index i was incremented in the wrong place, and the check on the presence of a next buffer in the chain was actually done for the next desc_status rather than the current one. Type: fix Signed-off-by: Mauro Sardara Change-Id: I74a64a34fea497900b7969cd96e1aeeb570a1bba --- src/plugins/memif/node.c | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) (limited to 'src/plugins/memif') diff --git a/src/plugins/memif/node.c b/src/plugins/memif/node.c index 5bb10fc6eec..ce9f95bc453 100644 --- a/src/plugins/memif/node.c +++ b/src/plugins/memif/node.c @@ -247,7 +247,7 @@ memif_process_desc (vlib_main_t *vm, vlib_node_runtime_t *node, u32 n_buffers = 0; u32 n_left = ptd->n_packets; u32 packet_len; - int i = 0; + int i = -1; /* construct copy and packet vector out of ring slots */ while (n_left) @@ -261,6 +261,7 @@ memif_process_desc (vlib_main_t *vm, vlib_node_runtime_t *node, dst_off = start_offset; next_slot: + i++; /* next descriptor */ n_bytes_left = desc_len[i]; packet_len += n_bytes_left; @@ -287,9 +288,6 @@ memif_process_desc (vlib_main_t *vm, vlib_node_runtime_t *node, } while (PREDICT_FALSE (n_bytes_left)); - /* next descriptor */ - i++; - if (desc_status[i].next) { src_off = 0; -- cgit 1.2.3-korg