buffers: revert protect against bad thread indices 41/37741/1
authorBenoît Ganne <bganne@cisco.com>
Fri, 2 Dec 2022 14:30:56 +0000 (15:30 +0100)
committerBenoît Ganne <bganne@cisco.com>
Fri, 2 Dec 2022 14:58:39 +0000 (15:58 +0100)
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>
src/vlib/buffer_funcs.c

index 80bb30e..f0e2add 100644 (file)
@@ -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;