diff options
| author |   Matthew Smith <mgsmith@netgate.com> | 2022-10-06 18:01:23 +0000 |
|---|---|---|
| committer |   Neale Ranns <neale@graphiant.com> | 2022-10-07 04:16:36 +0000 |
| commit | ba3d5afb92cf2c89e9b17d17a6668baf17260009 (patch) | |
| tree | b266a8307c2ee5ffe86dcbbac5251cfa01f6a277 /src/plugins/abf | |
| parent | 6ff8e90ed45e25e619f27beee1e0d016234e141e (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
Diffstat (limited to 'src/plugins/abf')
| -rw-r--r-- | src/plugins/abf/abf_api.c | 12 |
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); |