summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
-rw-r--r--src/vnet/config.c4
-rw-r--r--src/vnet/feature/feature.c6
2 files changed, 5 insertions, 5 deletions
diff --git a/src/vnet/config.c b/src/vnet/config.c
index 03189d77cd5..26b0cad6055 100644
--- a/src/vnet/config.c
+++ b/src/vnet/config.c
@@ -247,7 +247,7 @@ vnet_config_add_feature (vlib_main_t * vm,
u32 node_index = vec_elt (cm->node_index_by_feature_index, feature_index);
if (node_index == ~0) // feature node does not exist
- return config_string_heap_index; // return original config index
+ return ~0;
if (config_string_heap_index == ~0)
{
@@ -330,7 +330,7 @@ vnet_config_del_feature (vlib_main_t * vm,
/* Feature not found. */
if (f >= vec_end (old->features))
- return config_string_heap_index; // return original config index
+ return ~0;
new_features = duplicate_feature_vector (old->features);
f = new_features + (f - old->features);
diff --git a/src/vnet/feature/feature.c b/src/vnet/feature/feature.c
index 97100049b86..714e20e8e72 100644
--- a/src/vnet/feature/feature.c
+++ b/src/vnet/feature/feature.c
@@ -186,7 +186,7 @@ vnet_feature_enable_disable_with_index (u8 arc_index, u32 feature_index,
vnet_feature_main_t *fm = &feature_main;
vnet_feature_config_main_t *cm;
i16 feature_count;
- u32 old_ci, ci;
+ u32 ci;
if (arc_index == (u8) ~ 0)
return VNET_API_ERROR_INVALID_VALUE;
@@ -196,7 +196,7 @@ vnet_feature_enable_disable_with_index (u8 arc_index, u32 feature_index,
cm = &fm->feature_config_mains[arc_index];
vec_validate_init_empty (cm->config_index_by_sw_if_index, sw_if_index, ~0);
- old_ci = ci = cm->config_index_by_sw_if_index[sw_if_index];
+ ci = cm->config_index_by_sw_if_index[sw_if_index];
vec_validate (fm->feature_count_by_sw_if_index[arc_index], sw_if_index);
feature_count = fm->feature_count_by_sw_if_index[arc_index][sw_if_index];
@@ -209,7 +209,7 @@ vnet_feature_enable_disable_with_index (u8 arc_index, u32 feature_index,
: vnet_config_del_feature)
(vlib_get_main (), &cm->config_main, ci, feature_index, feature_config,
n_feature_config_bytes);
- if (old_ci == ci)
+ if (ci == ~0)
{
return 0;
}