> On Jun 9, 2015, at 3:29 PM, Ben Pfaff <b...@nicira.com> wrote:
> 
> On Tue, Jun 09, 2015 at 03:25:07PM -0700, Jarno Rajahalme wrote:
>> Commit () fixed the memory leak when a rule is deleted, but failed to
>> do the same when all rules in a bridge are deleted just before the
>> bridge itself is deleted.
>> 
>> This patch adds the necessary unref to ofproto_rule_delete().
>> 
>> Signed-off-by: Jarno Rajahalme <jrajaha...@nicira.com>
> 
> The commit number and subject is missing above.
> 

Oops, added.

> I assume you tested this?  Assuming you did,
> 

Yes. Easy to test, as extra unrefs crash fast. Hope this was not hiding other 
bugs, though.

> Acked-by: Ben Pfaff <b...@nicira.com>

Thanks for the review, pushed to master.

  Jarno

_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to