vtysh: return non-zero for configuration failures

[mirror_frr.git] / bgpd / bgp_bfd.c

diff --git a/bgpd/bgp_bfd.c b/bgpd/bgp_bfd.c
index 890acb3a4759f5808603962f39e02a9f6977596e..1aa2b93a90e442ca36082014e297c767c8bdabf7 100644 (file)
--- a/bgpd/bgp_bfd.c
+++ b/bgpd/bgp_bfd.c
@@ -562,7 +562,7 @@ DEFUN (neighbor_bfd,
 
   peer = peer_and_group_lookup_vty (vty, argv[idx_peer]->arg);
   if (! peer)
-    return CMD_WARNING;
+    return CMD_WARNING_CONFIG_FAILED;
 
   ret = bgp_bfd_peer_param_set (peer, BFD_DEF_MIN_RX, BFD_DEF_MIN_TX,
                                   BFD_DEF_DETECT_MULT, 1);
@@ -595,7 +595,7 @@ DEFUN (neighbor_bfd_param,
 
   peer = peer_and_group_lookup_vty (vty, argv[idx_peer]->arg);
   if (!peer)
-    return CMD_WARNING;
+    return CMD_WARNING_CONFIG_FAILED;
 
   if ((ret = bfd_validate_param (vty, argv[idx_number_1]->arg, argv[idx_number_2]->arg, argv[idx_number_3]->arg, &dm_val,
                                  &rx_val, &tx_val)) != CMD_SUCCESS)
@@ -626,14 +626,14 @@ DEFUN_HIDDEN (neighbor_bfd_type,
 
   peer = peer_and_group_lookup_vty (vty, argv[idx_peer]->arg);
   if (!peer)
-    return CMD_WARNING;
+    return CMD_WARNING_CONFIG_FAILED;
 
   if (strmatch(argv[idx_hop]->text, "singlehop"))
     type = BFD_TYPE_SINGLEHOP;
   else if (strmatch(argv[idx_hop]->text, "multihop"))
     type = BFD_TYPE_MULTIHOP;
   else
-    return CMD_WARNING;
+    return CMD_WARNING_CONFIG_FAILED;
 
   ret = bgp_bfd_peer_param_type_set (peer, type);
   if (ret != 0)
@@ -659,7 +659,7 @@ DEFUN (no_neighbor_bfd,
 
   peer = peer_and_group_lookup_vty (vty, argv[idx_peer]->arg);
   if (! peer)
-    return CMD_WARNING;
+    return CMD_WARNING_CONFIG_FAILED;
 
   ret = bgp_bfd_peer_param_unset(peer);
   if (ret != 0)
@@ -685,7 +685,7 @@ DEFUN_HIDDEN (no_neighbor_bfd_type,
 
   peer = peer_and_group_lookup_vty (vty, argv[idx_peer]->arg);
   if (! peer)
-    return CMD_WARNING;
+    return CMD_WARNING_CONFIG_FAILED;
 
   if (!peer->bfd_info)
     return 0;