tls: do not overwrite config parameters if set 28/36028/2
authorOfer Heifetz <oferh@marvell.com>
Thu, 14 Apr 2022 16:21:19 +0000 (19:21 +0300)
committerFlorin Coras <florin.coras@gmail.com>
Mon, 25 Apr 2022 18:21:38 +0000 (18:21 +0000)
User can define first_seg_size and add-segment-size in the tls
configuration section, but current code does not take this configuration
into account and sets a default value to first_seg_size and add_seg_size
respectively.

This commit checks if configuration was set and only if not uses the
default values.

Type: fix

Signed-off-by: Ofer Heifetz <oferh@marvell.com>
Change-Id: I0077f7d54fe7773dd92522476f882c924fda22df

src/vnet/tls/tls.c

index 9cf1008..6482d5d 100644 (file)
@@ -1315,8 +1315,11 @@ tls_init (vlib_main_t * vm)
   vec_validate (tm->rx_bufs, num_threads - 1);
   vec_validate (tm->tx_bufs, num_threads - 1);
 
-  tm->first_seg_size = 32 << 20;
-  tm->add_seg_size = 256 << 20;
+  if (!tm->first_seg_size)
+    tm->first_seg_size = 32 << 20;
+
+  if (!tm->add_seg_size)
+    tm->add_seg_size = 256 << 20;
 
   transport_register_protocol (TRANSPORT_PROTO_TLS, &tls_proto,
                               FIB_PROTOCOL_IP4, ~0);