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 <[email protected]> > Reported-at: http://openvswitch.org/pipermail/dev/2016-May/071381.html > Signed-off-by: Ben Pfaff <[email protected]> > --- > 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 [email protected] > Derek Cormier [email protected] > Dhaval Badiani [email protected] > DK Moon [email protected] > +Dong Zhi [email protected]
Please note that there is a typo here in the name of the author (“Ding”). > Edwin Chiu [email protected] > Eivind Bulie Haanaes > Enas Ahmad [email protected] > 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 <[email protected]> _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
