Re: [ovs-dev] [PATCH 2/2] ofproto: Don't destroy mutex before its use.

2014-04-24 Thread Ben Pfaff
On Tue, Apr 22, 2014 at 02:15:12PM -0700, Gurucharan Shetty wrote: > Currently, we are calling guarded_list_destroy() > to destroy a mutex and then go ahead and use it through > delete_group > ->delete_group__ >->handle_flow_mod__ > ->run_rule_executes > ->guarded_list_pop_all > > Th

[ovs-dev] [PATCH 2/2] ofproto: Don't destroy mutex before its use.

2014-04-22 Thread Gurucharan Shetty
Currently, we are calling guarded_list_destroy() to destroy a mutex and then go ahead and use it through delete_group ->delete_group__ ->handle_flow_mod__ ->run_rule_executes ->guarded_list_pop_all The group related unit tests cause ovs-vswitchd to crash because of this (on windows).