aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMatthew Smith <mgsmith@netgate.com>2022-10-06 18:01:23 +0000
committerNeale Ranns <neale@graphiant.com>2022-10-07 04:16:36 +0000
commitba3d5afb92cf2c89e9b17d17a6668baf17260009 (patch)
treeb266a8307c2ee5ffe86dcbbac5251cfa01f6a277
parent6ff8e90ed45e25e619f27beee1e0d016234e141e (diff)
abf: return status of attachment add/del
Type: fix The handler for abf_itf_attach_add_del was always returning 0. Set rv to the return value of call to abf_itf_attach() or abf_itf_detach(). Signed-off-by: Matthew Smith <mgsmith@netgate.com> Change-Id: Ibb888bb148e6e03fc2776e2384b3a6e26148a429
-rw-r--r--src/plugins/abf/abf_api.c12
1 files changed, 5 insertions, 7 deletions
diff --git a/src/plugins/abf/abf_api.c b/src/plugins/abf/abf_api.c
index 222e1f4e116..ffb32d582c5 100644
--- a/src/plugins/abf/abf_api.c
+++ b/src/plugins/abf/abf_api.c
@@ -113,16 +113,14 @@ vl_api_abf_itf_attach_add_del_t_handler (vl_api_abf_itf_attach_add_del_t * mp)
if (mp->is_add)
{
- abf_itf_attach (fproto,
- ntohl (mp->attach.policy_id),
- ntohl (mp->attach.priority),
- ntohl (mp->attach.sw_if_index));
+ rv = abf_itf_attach (fproto, ntohl (mp->attach.policy_id),
+ ntohl (mp->attach.priority),
+ ntohl (mp->attach.sw_if_index));
}
else
{
- abf_itf_detach (fproto,
- ntohl (mp->attach.policy_id),
- ntohl (mp->attach.sw_if_index));
+ rv = abf_itf_detach (fproto, ntohl (mp->attach.policy_id),
+ ntohl (mp->attach.sw_if_index));
}
REPLY_MACRO (VL_API_ABF_ITF_ATTACH_ADD_DEL_REPLY);