From ba3d5afb92cf2c89e9b17d17a6668baf17260009 Mon Sep 17 00:00:00 2001 From: Matthew Smith Date: Thu, 6 Oct 2022 18:01:23 +0000 Subject: [PATCH] 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 Change-Id: Ibb888bb148e6e03fc2776e2384b3a6e26148a429 --- src/plugins/abf/abf_api.c | 12 +++++------- 1 file 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); -- 2.16.6