ovs-brcompatd: Don't delete bridge twice.
authorBen Pfaff <blp@nicira.com>
Tue, 2 Mar 2010 22:19:13 +0000 (14:19 -0800)
committerBen Pfaff <blp@nicira.com>
Tue, 2 Mar 2010 22:19:13 +0000 (14:19 -0800)
commitc43f2a3282cb45742b03c0b550ee8e614f7db904
tree9ffc054655fca35bb122c4170671e708ec59304d
parent5a195c94ae4d663925a99b8dba761b9d961bdf9e
ovs-brcompatd: Don't delete bridge twice.

Somehow I totally goofed when I wrote commit dcdf169e "ovs-brcompatd:
Delete Bridge record when deleting a bridge."  In fact, del_bridge()
already deleted the Bridge record, and I missed that, and so I added a
*second* call to ovsrec_bridge_delete(br).  Obviously that's bad.

This commit removes the first call to delete the bridge.  It is a slight
improvement over simply reverting the buggy commit in that it doesn't
refer to 'br' (just as a pointer equality comparison) after deleting it.

Bug #2448.
vswitchd/ovs-brcompatd.c