summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorFlorin Coras <fcoras@cisco.com>2023-02-02 20:07:19 -0800
committerDave Wallace <dwallacelf@gmail.com>2023-02-03 20:11:51 +0000
commitaf2e88d964ff8bd8eca30c182ab00e178213c1d4 (patch)
tree3eddc479199b5f2eb6bae3facf9f118517f33307
parent6b97c43005f6458ce2e253f87af6f609eaebef60 (diff)
session: fix out of bounds event memcpy
Type: fix Signed-off-by: Florin Coras <fcoras@cisco.com> Change-Id: If5300653edd2dad470985f4591959d00cad2a43b
-rw-r--r--src/vnet/session/session_debug.c4
1 files changed, 1 insertions, 3 deletions
diff --git a/src/vnet/session/session_debug.c b/src/vnet/session/session_debug.c
index a6c4adb2bdc..dba4ef36fb8 100644
--- a/src/vnet/session/session_debug.c
+++ b/src/vnet/session/session_debug.c
@@ -302,7 +302,6 @@ session_node_lookup_fifo_event (svm_fifo_t * f, session_event_t * e)
session_worker_t *wrk;
int i, index, found = 0;
svm_msg_q_msg_t *msg;
- svm_msg_q_ring_t *ring;
svm_msg_q_t *mq;
u8 thread_index;
@@ -319,8 +318,7 @@ session_node_lookup_fifo_event (svm_fifo_t * f, session_event_t * e)
for (i = 0; i < sq->cursize; i++)
{
msg = (svm_msg_q_msg_t *) (&sq->data[0] + sq->elsize * index);
- ring = svm_msg_q_ring (mq, msg->ring_index);
- clib_memcpy_fast (e, svm_msg_q_msg_data (mq, msg), ring->elsize);
+ clib_memcpy_fast (e, svm_msg_q_msg_data (mq, msg), sizeof (*e));
found = session_node_cmp_event (e, f);
if (found)
return 1;