From: jiangxiaoming <jiangxiaoming@outlook.com>
Date: Thu, 17 Dec 2020 02:35:16 +0000 (+0800)
Subject: hsa: http server: memset after pool_put will cause ASAN issue
X-Git-Tag: v21.10-rc0~790
X-Git-Url: https://gerrit.fd.io/r/gitweb?a=commitdiff_plain;h=5f9a5c70edd11e90a27a795d953e06606424d4de;p=vpp.git

hsa: http server: memset after pool_put will cause ASAN issue

Type: fix
Signed-off-by: jiangxiaoming <jiangxiaoming@outlook.com>
Change-Id: I3783f72ff93c362589e859f389d94eadd2aadddb
---

diff --git a/src/plugins/hs_apps/http_server.c b/src/plugins/hs_apps/http_server.c
index a93261f0b04..ce4e09addbc 100644
--- a/src/plugins/hs_apps/http_server.c
+++ b/src/plugins/hs_apps/http_server.c
@@ -136,9 +136,10 @@ static void
 http_server_session_free (http_session_t * hs)
 {
   http_server_main_t *hsm = &http_server_main;
-  pool_put (hsm->sessions[hs->thread_index], hs);
+  u32 thread = hs->thread_index;
   if (CLIB_DEBUG)
     memset (hs, 0xfa, sizeof (*hs));
+  pool_put (hsm->sessions[thread], hs);
 }
 
 static void