svn commit: trunk/busybox/networking: libiproute

vda at busybox.net vda at busybox.net
Sat Jan 27 13:13:49 UTC 2007


Author: vda
Date: 2007-01-27 05:13:45 -0800 (Sat, 27 Jan 2007)
New Revision: 17550

Log:
fix numbering in switch(): we don't have "del" anymore,
substring match on "delete" handles it ok. But we (I?)
forgot to renumber case labels...


Modified:
   trunk/busybox/networking/ip.c
   trunk/busybox/networking/libiproute/iproute.c


Changeset:
Modified: trunk/busybox/networking/ip.c
===================================================================
--- trunk/busybox/networking/ip.c	2007-01-27 08:19:09 UTC (rev 17549)
+++ trunk/busybox/networking/ip.c	2007-01-27 13:13:45 UTC (rev 17550)
@@ -33,8 +33,9 @@
 		if (ENABLE_FEATURE_IP_LINK && matches(argv[1], "link") == 0) {
 			ret = do_iplink(argc-2, argv+2);
 		}
-		if (ENABLE_FEATURE_IP_TUNNEL &&
-			(matches(argv[1], "tunnel") == 0 || strcmp(argv[1], "tunl") == 0)) {
+		if (ENABLE_FEATURE_IP_TUNNEL
+		 && (matches(argv[1], "tunnel") == 0 || strcmp(argv[1], "tunl") == 0)
+		) {
 			ret = do_iptunnel(argc-2, argv+2);
 		}
 		if (ENABLE_FEATURE_IP_RULE && matches(argv[1], "rule") == 0) {

Modified: trunk/busybox/networking/libiproute/iproute.c
===================================================================
--- trunk/busybox/networking/libiproute/iproute.c	2007-01-27 08:19:09 UTC (rev 17549)
+++ trunk/busybox/networking/libiproute/iproute.c	2007-01-27 13:13:45 UTC (rev 17550)
@@ -835,9 +835,11 @@
 
 int do_iproute(int argc, char **argv)
 {
-	static const char * const ip_route_commands[] =
-		{ "add", "append", "change", "chg", "delete", "get",
-		"list", "show", "prepend", "replace", "test", "flush", 0 };
+	static const char * const ip_route_commands[] = {
+	/*0-3*/	"add", "append", "change", "chg",
+	/*4-7*/	"delete", "get", "list", "show",
+	/*8..*/	"prepend", "replace", "test", "flush", 0
+	};
 	int command_num = 6;
 	unsigned int flags = 0;
 	int cmd = RTM_NEWROUTE;
@@ -848,7 +850,7 @@
 		command_num = index_in_substr_array(ip_route_commands, *argv);
 	}
 	switch (command_num) {
-		case 0: /* add*/
+		case 0: /* add */
 			flags = NLM_F_CREATE|NLM_F_EXCL;
 			break;
 		case 1: /* append */
@@ -859,21 +861,20 @@
 			flags = NLM_F_REPLACE;
 			break;
 		case 4: /* delete */
-		case 5: /* del */
 			cmd = RTM_DELROUTE;
 			break;
-		case 6: /* get */
+		case 5: /* get */
 			return iproute_get(argc-1, argv+1);
-		case 7: /* list */
-		case 8: /* show */
+		case 6: /* list */
+		case 7: /* show */
 			return iproute_list_or_flush(argc-1, argv+1, 0);
-		case 9: /* prepend */
+		case 8: /* prepend */
 			flags = NLM_F_CREATE;
-		case 10: /* replace */
+		case 9: /* replace */
 			flags = NLM_F_CREATE|NLM_F_REPLACE;
-		case 11: /* test */
+		case 10: /* test */
 			flags = NLM_F_EXCL;
-		case 12: /* flush */
+		case 11: /* flush */
 			return iproute_list_or_flush(argc-1, argv+1, 1);
 		default:
 			bb_error_msg_and_die("unknown command %s", *argv);




More information about the busybox-cvs mailing list