BIER coverity Fix 79/9779/2
authorNeale Ranns <nranns@cisco.com>
Sun, 10 Dec 2017 13:32:31 +0000 (05:32 -0800)
committerFlorin Coras <florin.coras@gmail.com>
Sun, 10 Dec 2017 20:29:47 +0000 (20:29 +0000)
Change-Id: I17f95775756b8fd5ee390432f6d8041abbd9f974
Signed-off-by: Neale Ranns <nranns@cisco.com>
src/vnet/bier/bier_test.c

index cf75c2e..0fbca7e 100644 (file)
@@ -170,15 +170,17 @@ bier_test_validate_entry (index_t bei,
 
     bier_entry_contribute_forwarding(bei, &dpo);
 
-    BIER_TEST_LB((DPO_LOAD_BALANCE == dpo.dpoi_type),
-                 "Entry links to %U",
-                 format_dpo_type, dpo.dpoi_type);
+    res = BIER_TEST_I((DPO_LOAD_BALANCE == dpo.dpoi_type),
+                      "Entry links to %U",
+                      format_dpo_type, dpo.dpoi_type);
 
-    lb = load_balance_get(dpo.dpoi_index);
-    res = fib_test_validate_lb_v(lb, n_buckets, &ap);
+    if (res)
+    {
+        lb = load_balance_get(dpo.dpoi_index);
+        res = fib_test_validate_lb_v(lb, n_buckets, &ap);
+    }
 
     dpo_reset(&dpo);
-
     va_end(ap);
 
     return (res);