lb: refactor ntohl in lb_conf handler 82/22182/5
authorVratko Polak <vrpolak@cisco.com>
Fri, 20 Sep 2019 16:56:49 +0000 (18:56 +0200)
committerPaul Vinciguerra <pvinci@vinciconsulting.com>
Wed, 30 Oct 2019 00:36:02 +0000 (00:36 +0000)
Type: refactor

Change-Id: Ibe8f3c1ed5c04d1800fea9b9a90772816aa38d65
Signed-off-by: Vratko Polak <vrpolak@cisco.com>
src/plugins/lb/api.c

index cafa212..67a26d5 100644 (file)
@@ -48,19 +48,19 @@ vl_api_lb_conf_t_handler
 {
   lb_main_t *lbm = &lb_main;
   vl_api_lb_conf_reply_t * rmp;
+  u32 sticky_buckets_per_core, flow_timeout;
   int rv = 0;
 
-  if (mp->sticky_buckets_per_core == ~0) {
-    mp->sticky_buckets_per_core = htonl(lbm->per_cpu_sticky_buckets);
-  }
-  if (mp->flow_timeout == ~0) {
-    mp->flow_timeout = htonl(lbm->flow_timeout);
-  }
+  sticky_buckets_per_core = mp->sticky_buckets_per_core == ~0
+                           ? lbm->per_cpu_sticky_buckets
+                           : ntohl(mp->sticky_buckets_per_core);
+  flow_timeout = mp->flow_timeout == ~0
+                ? lbm->flow_timeout
+                : ntohl(mp->flow_timeout);
 
   rv = lb_conf((ip4_address_t *)&mp->ip4_src_address,
-               (ip6_address_t *)&mp->ip6_src_address,
-               ntohl(mp->sticky_buckets_per_core),
-               ntohl(mp->flow_timeout));
+              (ip6_address_t *)&mp->ip6_src_address,
+              sticky_buckets_per_core, flow_timeout);
 
  REPLY_MACRO (VL_API_LB_CONF_REPLY);
 }