summaryrefslogtreecommitdiffstats
path: root/src/vnet/fib/fib_test.c
diff options
context:
space:
mode:
authorPaul Vinciguerra <pvinci@vinciconsulting.com>2018-10-24 12:14:09 -0700
committerNeale Ranns <nranns@cisco.com>2018-10-25 14:30:49 +0000
commit0c0383df88836b5e072f63c3b3a8f5f6b24c094b (patch)
treecbe6f73e3de450b279d4c7b956771477e0570ff1 /src/vnet/fib/fib_test.c
parent2bfb9e15d8ed7d040de7e453d764a6eb79668f5f (diff)
Trivial: Cleanup missing va_ends.
Change-Id: Ie7827b6a31968a355687d27325c0f30cab1bc890 Signed-off-by: Paul Vinciguerra <pvinci@vinciconsulting.com>
Diffstat (limited to 'src/vnet/fib/fib_test.c')
-rw-r--r--src/vnet/fib/fib_test.c11
1 files changed, 7 insertions, 4 deletions
diff --git a/src/vnet/fib/fib_test.c b/src/vnet/fib/fib_test.c
index 492369c3540..e26cc674a88 100644
--- a/src/vnet/fib/fib_test.c
+++ b/src/vnet/fib/fib_test.c
@@ -212,7 +212,8 @@ fib_test_mk_intf (u32 ninterfaces)
#define FIB_TEST_RPF(_cond, _comment, _args...) \
{ \
if (FIB_TEST_I(_cond, _comment, ##_args)) { \
- return (1); \
+ res = 1; \
+ goto cleanup; \
} \
}
@@ -258,6 +259,7 @@ fib_test_urpf_is_equal (fib_node_index_t fei,
dpo_reset(&dpo);
+cleanup:
va_end(ap);
return (res);
@@ -671,7 +673,6 @@ fib_test_validate_entry (fib_node_index_t fei,
va_list ap;
int res;
- va_start(ap, n_buckets);
res = 0;
pfx = fib_entry_get_prefix(fei);
@@ -682,8 +683,10 @@ fib_test_validate_entry (fib_node_index_t fei,
{
const replicate_t *rep;
+ va_start(ap, n_buckets);
rep = replicate_get(dpo.dpoi_index);
res = fib_test_validate_rep_v(rep, n_buckets, &ap);
+ va_end (ap);
}
else
{
@@ -694,8 +697,10 @@ fib_test_validate_entry (fib_node_index_t fei,
format_fib_prefix, pfx,
format_dpo_type, dpo.dpoi_type);
+ va_start(ap, n_buckets);
lb = load_balance_get(dpo.dpoi_index);
res = fib_test_validate_lb_v(lb, n_buckets, &ap);
+ va_end(ap);
/*
* ensure that the LB contributed by the entry is the
@@ -736,8 +741,6 @@ fib_test_validate_entry (fib_node_index_t fei,
dpo_reset(&dpo);
- va_end(ap);
-
return (res);
}