diff options
| author |   Neale Ranns <nranns@cisco.com> | 2017-11-15 10:44:07 -0800 |
|---|---|---|
| committer |   Chris Luke <chris_luke@comcast.com> | 2017-11-15 19:45:47 +0000 |
| commit | 93149bb4cd0a137b2a6da4b0d9f8cb2032e44734 (patch) | |
| tree | 5818b346974d496f784050220ffca9a86052df4f /src/vnet/fib/fib_test.c | |
| parent | b8a0d2cf9ff8796123b3c167c051f78ab03cc4cf (diff) | |
BIER: coverity fixes
Change-Id: I657bade082f9f754b294cd5f23ecfad4f0f46265
Signed-off-by: Neale Ranns <nranns@cisco.com>
Diffstat (limited to 'src/vnet/fib/fib_test.c')
| -rw-r--r-- | src/vnet/fib/fib_test.c | 16 |
1 files changed, 11 insertions, 5 deletions
diff --git a/src/vnet/fib/fib_test.c b/src/vnet/fib/fib_test.c index b74ec337c1f..555196d0092 100644 --- a/src/vnet/fib/fib_test.c +++ b/src/vnet/fib/fib_test.c @@ -612,12 +612,18 @@ fib_test_validate_lb (const dpo_id_t *dpo, va_start(ap, n_buckets); - FIB_TEST_LB((DPO_LOAD_BALANCE == dpo->dpoi_type), - "Entry links to %U", - format_dpo_type, dpo->dpoi_type); - lb = load_balance_get(dpo->dpoi_index); + if (FIB_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); + res = fib_test_validate_lb_v(lb, n_buckets, &ap); + } + else + { + res = !0; + } va_end(ap); |