]> git.puffer.fish Git - matthieu/frr.git/commitdiff
bgpd: Remove redundant warning message
authorDonald Sharp <sharpd@cumulusnetworks.com>
Wed, 25 Jan 2017 14:43:11 +0000 (09:43 -0500)
committerDonald Sharp <sharpd@cumulusnetworks.com>
Wed, 25 Jan 2017 14:43:11 +0000 (09:43 -0500)
When bgp_vty_find_and_parse_afi_safi_vrf detects
a invalidly named vrf, it warns the user.  There
is no need for the calling function to warn
again.

Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>
bgpd/bgp_route.c

index 92a2843ba977e7b684c9eed3f08b1253dbb3efe4..02ba731f04e4187217a2483256f1187c1e9442be 100644 (file)
@@ -7877,10 +7877,8 @@ DEFUN (show_ip_bgp,
 
   idx = bgp_vty_find_and_parse_afi_safi_vrf (vty, argv, argc, idx, &afi, &safi, &vrf);
   if (!idx)
-    {
-      vty_out (vty, "View/Vrf Specified: %s is unknown", argv[5]->arg);
-      return CMD_WARNING;
-    }
+    return CMD_WARNING;
+
   int uj = use_json (argc, argv);
   if (uj) argc--;
 
@@ -7986,10 +7984,7 @@ DEFUN (show_ip_bgp_route,
 
   idx = bgp_vty_find_and_parse_afi_safi_vrf (vty, argv, argc, idx, &afi, &safi, &vrf);
   if (!idx)
-    {
-      vty_out (vty, "View/Vrf Specified: %s is unknown", argv[5]->arg);
-      return CMD_WARNING;
-    }
+    return CMD_WARNING;
 
   if (vrf != VRF_ALL)
     {
@@ -8056,10 +8051,7 @@ DEFUN (show_ip_bgp_regexp,
   int idx = 0;
   idx = bgp_vty_find_and_parse_afi_safi_vrf (vty, argv, argc, idx, &afi, &safi, &vrf);
   if (!idx)
-    {
-      vty_out (vty, "View/Vrf Specified: %s is unknown", argv[5]->arg);
-      return CMD_WARNING;
-    }
+    return CMD_WARNING;
 
   // get index of regex
   argv_find (argv, argc, "regexp", &idx);
@@ -8093,10 +8085,8 @@ DEFUN (show_ip_bgp_instance_all,
   int idx = 0;
   idx = bgp_vty_find_and_parse_afi_safi_vrf (vty, argv, argc, idx, &afi, &safi, &vrf);
   if (!idx)
-    {
-      vty_out (vty, "View/Vrf Specified: %s is unknown", argv[5]->arg);
-      return CMD_WARNING;
-    }
+    return CMD_WARNING;
+
   int uj = use_json (argc, argv);
   if (uj) argc--;
 
@@ -8786,10 +8776,8 @@ DEFUN (show_ip_bgp_instance_neighbor_prefix_counts,
 
   idx = bgp_vty_find_and_parse_afi_safi_vrf (vty, argv, argc, idx, &afi, &safi, &vrf);
   if (!idx)
-    {
-      vty_out (vty, "View/Vrf Specified: %s is unknown", argv[5]->arg);
-      return CMD_WARNING;
-    }
+    return CMD_WARNING;
+
   int uj = use_json (argc, argv);
   if (uj) argc--;
 
@@ -9146,10 +9134,8 @@ DEFUN (show_ip_bgp_instance_neighbor_advertised_route,
 
   idx = bgp_vty_find_and_parse_afi_safi_vrf (vty, argv, argc, idx, &afi, &safi, &vrf);
   if (!idx)
-    {
-      vty_out (vty, "View/Vrf Specified: %s is unknown", argv[5]->arg);
-      return CMD_WARNING;
-    }
+    return CMD_WARNING;
+
   int uj = use_json (argc, argv);
   if (uj) argc--;
 
@@ -9334,12 +9320,10 @@ DEFUN (show_ip_bgp_neighbor_routes,
 
   int idx = 0;
 
-    idx = bgp_vty_find_and_parse_afi_safi_vrf (vty, argv, argc, idx, &afi, &safi, &vrf);
+  idx = bgp_vty_find_and_parse_afi_safi_vrf (vty, argv, argc, idx, &afi, &safi, &vrf);
   if (!idx)
-    {
-      vty_out (vty, "View/Vrf Specified: %s is unknown", argv[5]->arg);
-      return CMD_WARNING;
-    }
+    return CMD_WARNING;
+
   int uj = use_json (argc, argv);
   if (uj) argc--;