From 6019f124107ff8f86ecc1dc65fde64cf935ac7d8 Mon Sep 17 00:00:00 2001 From: Ansis Atteka Date: Tue, 21 Aug 2012 16:48:36 -0700 Subject: [PATCH] stp: port_no counter is off by one This counter was off by one, because port_num should be less than STP_MAX_PORTS. This caused an assert hit later in stp_get_port(). Issue: 13059 Signed-off-by: Ansis Atteka Reported-by: Ram Jothikumar Acked-by: Ethan Jackson --- vswitchd/bridge.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/vswitchd/bridge.c b/vswitchd/bridge.c index bd8e772af..1fbbe32ac 100644 --- a/vswitchd/bridge.c +++ b/vswitchd/bridge.c @@ -937,7 +937,7 @@ port_configure_stp(const struct ofproto *ofproto, struct port *port, bitmap_set1(port_num_bitmap, port_idx); port_s->port_num = port_idx; } else { - if (*port_num_counter > STP_MAX_PORTS) { + if (*port_num_counter >= STP_MAX_PORTS) { VLOG_ERR("port %s: too many STP ports, disabling", port->name); port_s->enable = false; return; -- 2.43.0