[ovs-git] Open vSwitch: ofproto-dpif-xlate: Fix confusion between "no stp port" and "stp port 0". (branch-2.0)

dev at openvswitch.org dev at openvswitch.org
Wed Sep 4 22:44:03 UTC 2013


This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "Open vSwitch".

The branch, branch-2.0 has been updated
       via  c9dad52a818693e5d5eb75891db9e6f0a5053886 (commit)
      from  e374f18644b62bf893d1082f2965c33978c01e46 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit c9dad52a818693e5d5eb75891db9e6f0a5053886
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=c9dad52a818693e5d5eb75891db9e6f0a5053886
Author: Alex Wang <alexw at nicira.com>
		
ofproto-dpif-xlate: Fix confusion between "no stp port" and "stp port 0".
		
Commit 9d189a50e (ofproto-dpif-xlate: Pull STP xlation into
ofproto-dpif-xlate.) introduced the bug that considers 'stp_port_no'
of 0 as stp disabled on the port.  However 'stp_port_no' is
actually the index of the stp struct's port array and ranges
between [0, STP_MAX_PORTS).  So the bug allows the blocked
port keep transmitting packets and generates loop.

This commit fixes this bug.

Signed-off-by: Alex Wang <alexw at nicira.com>
Signed-off-by: Ben Pfaff <blp at nicira.com>


-----------------------------------------------------------------------

Summary of changes:
 ofproto/ofproto-dpif-xlate.c |    4 ++--
 ofproto/ofproto-dpif.c       |    2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)


hooks/post-receive
-- 
Open vSwitch



More information about the git mailing list