From: Nathan Skrzypczak <nathan.skrzypczak@gmail.com>
Date: Mon, 19 Aug 2019 15:49:56 +0000 (+0200)
Subject: quic: fix rx_callback refactoring
X-Git-Tag: v20.05-rc0~973
X-Git-Url: https://gerrit.fd.io/r/gitweb?p=vpp.git;a=commitdiff_plain;h=1682b51c416561b083d84194d49ff617e070eb47

quic: fix rx_callback refactoring

* check_quic_client_connected might allocate ctx
and invalidate our pointer

Type: fix

Change-Id: I885ca5a1a6db9a7765c9047a7df9cdf66c94f1cb
Signed-off-by: Nathan Skrzypczak <nathan.skrzypczak@gmail.com>
---

diff --git a/src/plugins/quic/quic.c b/src/plugins/quic/quic.c
index 8abb850084d..6bf0e5eb726 100644
--- a/src/plugins/quic/quic.c
+++ b/src/plugins/quic/quic.c
@@ -2053,10 +2053,10 @@ quic_app_rx_callback (session_t * udp_session)
 	  if (packets_ctx[i].thread_index != thread_index)
 	    continue;
 
+	  check_quic_client_connected (&packets_ctx[i]);
 	  ctx =
 	    quic_ctx_get (packets_ctx[i].ctx_index,
 			  packets_ctx[i].thread_index);
-	  check_quic_client_connected (&packets_ctx[i]);
 	  quic_send_packets (ctx);
 	}
       svm_fifo_dequeue_drop (f, fifo_offset);