ipsec: Add/Del SA not MP safe 00/26800/2
authorNeale Ranns <nranns@cisco.com>
Thu, 30 Apr 2020 13:30:57 +0000 (13:30 +0000)
committerDave Wallace <dwallacelf@gmail.com>
Thu, 7 May 2020 01:00:15 +0000 (01:00 +0000)
Type: fix

some crytto engines store key data indexed by SA index. Creating new SAs
means this store reallocs with packets inflight; bad stuff ensues.

Signed-off-by: Neale Ranns <nranns@cisco.com>
Change-Id: Ia23c3a59e2d05fb006bdbd9922d01ee192e22853

src/vnet/ipsec/ipsec_api.c

index 4252acd..b86de04 100644 (file)
@@ -990,13 +990,6 @@ ipsec_api_hookup (vlib_main_t * vm)
   foreach_vpe_api_msg;
 #undef _
 
-  /*
-   * Adding and deleting SAs is MP safe since when they are added/delete
-   * no traffic is using them
-   */
-  am->is_mp_safe[VL_API_IPSEC_SAD_ENTRY_ADD_DEL] = 1;
-  am->is_mp_safe[VL_API_IPSEC_SAD_ENTRY_ADD_DEL_REPLY] = 1;
-
   /*
    * Set up the (msg_name, crc, message-id) table
    */