Hello!

"brctl stp br0" would crash brctl provided that br0 is a bridge. This is caused by incorrect number of arguments in the commands table. "stp" needs at least 2 arguments after "stp". The patch against version 1.0.4 is attached.

--
Regards,
Pavel Roskin
--- brctl/brctl_cmd.c
+++ brctl/brctl_cmd.c
@@ -394,7 +394,7 @@ static const struct command commands[] =
          "<bridge>\t\tshow a list of mac addrs"},
        { 1, "showstp", br_cmd_showstp, 
          "<bridge>\t\tshow bridge stp info"},
-       { 1, "stp", br_cmd_stp,
+       { 2, "stp", br_cmd_stp,
          "<bridge> <state>\tturn stp on/off" },
 };
 
_______________________________________________
Bridge mailing list
[email protected]
http://lists.osdl.org/mailman/listinfo/bridge

Reply via email to