summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorBenoît Ganne <bganne@cisco.com>2022-12-02 15:30:56 +0100
committerBenoît Ganne <bganne@cisco.com>2022-12-02 15:58:39 +0100
commitb01efc557b411e02379c9647eebf7e762efd8473 (patch)
tree9ad42193639dfc62bdbb587d3b32cc89d712a1a4
parentea0b890cbfb4a1a1b767516c935395190b546911 (diff)
buffers: revert protect against bad thread indices
This change was introduced to workaround a bug in the NAT code, but we should not woraround plugin bugs in infra. Type: fix Fixes: f8631ce7e8886136b4543a7926ffdf1bc760fb11 Change-Id: Id6ee281cf1fe8466b6522905fc2a176716e3d52f Signed-off-by: Benoît Ganne <bganne@cisco.com>
-rw-r--r--src/vlib/buffer_funcs.c5
1 files changed, 1 insertions, 4 deletions
diff --git a/src/vlib/buffer_funcs.c b/src/vlib/buffer_funcs.c
index 80bb30e9f93..f0e2add8457 100644
--- a/src/vlib/buffer_funcs.c
+++ b/src/vlib/buffer_funcs.c
@@ -261,10 +261,7 @@ vlib_get_frame_queue_elt (vlib_frame_queue_main_t *fqm, u32 index,
vlib_frame_queue_t *fq;
u64 nelts, tail, new_tail;
- if (index >= vec_len (fqm->vlib_frame_queues))
- return 0;
-
- fq = fqm->vlib_frame_queues[index];
+ fq = vec_elt (fqm->vlib_frame_queues, index);
ASSERT (fq);
nelts = fq->nelts;