summaryrefslogtreecommitdiffstats
path: root/src/vnet
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
parent2bfb9e15d8ed7d040de7e453d764a6eb79668f5f (diff)
Trivial: Cleanup missing va_ends.
Change-Id: Ie7827b6a31968a355687d27325c0f30cab1bc890 Signed-off-by: Paul Vinciguerra <pvinci@vinciconsulting.com>
Diffstat (limited to 'src/vnet')
-rw-r--r--src/vnet/fib/fib_table.c3
-rw-r--r--src/vnet/fib/fib_test.c11
-rw-r--r--src/vnet/mfib/mfib_test.c4
3 files changed, 11 insertions, 7 deletions
diff --git a/src/vnet/fib/fib_table.c b/src/vnet/fib/fib_table.c
index 69746d96912..f46a5b42183 100644
--- a/src/vnet/fib/fib_table.c
+++ b/src/vnet/fib/fib_table.c
@@ -1148,7 +1148,6 @@ fib_table_create_and_lock (fib_protocol_t proto,
fib_node_index_t fi;
va_list ap;
- va_start(ap, fmt);
switch (proto)
{
@@ -1167,6 +1166,8 @@ fib_table_create_and_lock (fib_protocol_t proto,
fib_table = fib_table_get(fi, proto);
+ va_start(ap, fmt);
+
fib_table->ft_desc = va_format(fib_table->ft_desc, fmt, &ap);
va_end(ap);
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);
}
diff --git a/src/vnet/mfib/mfib_test.c b/src/vnet/mfib/mfib_test.c
index 2a12a38ef80..fda02580bee 100644
--- a/src/vnet/mfib/mfib_test.c
+++ b/src/vnet/mfib/mfib_test.c
@@ -229,7 +229,6 @@ mfib_test_entry (fib_node_index_t fei,
va_list ap;
int res;
- va_start(ap, n_buckets);
res = 0;
mfe = mfib_entry_get(fei);
@@ -263,12 +262,13 @@ mfib_test_entry (fib_node_index_t fei,
format_mfib_prefix, &pfx,
format_dpo_type, tmp.dpoi_type);
+ va_start(ap, n_buckets);
res = mfib_test_validate_rep_v(rep, n_buckets, &ap);
+ va_end(ap);
dpo_reset(&tmp);
}
- va_end(ap);
return (res);
}