diff options
| author | Rafael Zalamena <rzalamena@opensourcerouting.org> | 2024-07-31 22:16:54 -0300 |
|---|---|---|
| committer | Rafael Zalamena <rzalamena@opensourcerouting.org> | 2024-09-30 11:13:25 -0300 |
| commit | e1338f5ade1fd223e8eaca750123894cc5e7ca03 (patch) | |
| tree | 872a0dabae4ef91b59728ba4c6ad54b5f7f63e93 /pimd | |
| parent | 30eb4b73ff0082f419c34e0a65636179aa02d9be (diff) | |
pimd: fix northbound error message on delete
`snprintf` doesn't know about `%pPAs` use `snprintfrr` instead.
Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>
Diffstat (limited to 'pimd')
| -rw-r--r-- | pimd/pim_nb_config.c | 27 |
1 files changed, 12 insertions, 15 deletions
diff --git a/pimd/pim_nb_config.c b/pimd/pim_nb_config.c index f864ce8f8e..ea8b56fee3 100644 --- a/pimd/pim_nb_config.c +++ b/pimd/pim_nb_config.c @@ -934,10 +934,9 @@ int routing_control_plane_protocols_control_plane_protocol_pim_address_family_ss yang_dnode_get_pimaddr(&source_addr, args->dnode, NULL); result = pim_ssmpingd_start(pim, source_addr); if (result) { - snprintf( - args->errmsg, args->errmsg_len, - "%% Failure starting ssmpingd for source %pPA: %d", - &source_addr, result); + snprintfrr(args->errmsg, args->errmsg_len, + "%% Failure starting ssmpingd for source %pPA: %d", &source_addr, + result); return NB_ERR_INCONSISTENCY; } } @@ -964,10 +963,9 @@ int routing_control_plane_protocols_control_plane_protocol_pim_address_family_ss yang_dnode_get_pimaddr(&source_addr, args->dnode, NULL); result = pim_ssmpingd_stop(pim, source_addr); if (result) { - snprintf( - args->errmsg, args->errmsg_len, - "%% Failure stopping ssmpingd for source %pPA: %d", - &source_addr, result); + snprintfrr(args->errmsg, args->errmsg_len, + "%% Failure stopping ssmpingd for source %pPA: %d", &source_addr, + result); return NB_ERR_INCONSISTENCY; } @@ -3860,10 +3858,9 @@ int lib_interface_gmp_address_family_join_group_destroy( GM_JOIN_STATIC); if (result) { - snprintf(args->errmsg, args->errmsg_len, - "%% Failure leaving " GM - " group %pPAs %pPAs on interface %s: %d", - &source_addr, &group_addr, ifp->name, result); + snprintfrr(args->errmsg, args->errmsg_len, + "%% Failure leaving " GM " group %pPAs %pPAs on interface %s: %d", + &source_addr, &group_addr, ifp->name, result); return NB_ERR_INCONSISTENCY; } @@ -3952,9 +3949,9 @@ int lib_interface_gmp_address_family_static_group_destroy( result = pim_if_static_group_del(ifp, group_addr, source_addr); if (result) { - snprintf(args->errmsg, args->errmsg_len, - "%% Failure removing static group %pPAs %pPAs on interface %s: %d", - &source_addr, &group_addr, ifp->name, result); + snprintfrr(args->errmsg, args->errmsg_len, + "%% Failure removing static group %pPAs %pPAs on interface %s: %d", + &source_addr, &group_addr, ifp->name, result); return NB_ERR_INCONSISTENCY; } |
