On Fri, May 20, 2016 at 05:30:25PM +0200, Quentin Monnet wrote: > Hi Ben, > > 2016-05-20 (07:49 -0700) ~ Ben Pfaff > > The stp/tcn command, which locks the mutex, was being registered without > > initializing the mutex, so calling stp/tcn before STP was enabled on the > > switch caused a crash. This commit fixes the bug by initializing the mutex > > at the same time we register the stp/tcn command. > > > > Reported-by: Ding Zhi <zhi.d...@6wind.com> > > Reported-at: http://openvswitch.org/pipermail/dev/2016-May/071381.html > > Signed-off-by: Ben Pfaff <b...@ovn.org> > > --- > > AUTHORS | 1 + > > lib/stp.c | 27 +++++++++++++++------------ > > 2 files changed, 16 insertions(+), 12 deletions(-) > > > > diff --git a/AUTHORS b/AUTHORS > > index 39ac60b..e39c2f0 100644 > > --- a/AUTHORS > > +++ b/AUTHORS > > @@ -300,6 +300,7 @@ David Palma pa...@onesource.pt > > Derek Cormier derek.corm...@lab.ntt.co.jp > > Dhaval Badiani dbadi...@vmware.com > > DK Moon dkm...@nicira.com > > +Dong Zhi zhi.d...@6wind.com > > Please note that there is a typo here in the name of the author (“Ding”).
Oops. Fixed. > > Edwin Chiu ec...@vmware.com > > Eivind Bulie Haanaes > > Enas Ahmad enas.ah...@kaust.edu.sa > > diff --git a/lib/stp.c b/lib/stp.c > > index 0f92ed1..ecef012 100644 > > […] > > This patch does address the issue of my previous email ( > http://openvswitch.org/pipermail/dev/2016-May/071381.html ). Thanks! > > Test-by: Quentin Monnet <quentin.mon...@6wind.com> Thanks, I applied this to master, branch-2.5, branch-2.4, and branch-2.3. _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
