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

dev at openvswitch.org dev at openvswitch.org
Wed Sep 4 22:20:23 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, master has been updated
       via  92cf817bad93a29079788ebbc233a8df0afca241 (commit)
      from  b09265a6d4deaab2749ed135f0a8a1d45f0b9379 (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 92cf817bad93a29079788ebbc233a8df0afca241
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=92cf817bad93a29079788ebbc233a8df0afca241
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