diff mbox

[ovs-dev,2/2] mcast-snooping: Avoid segfault for vswitchd.

Message ID 1488533841-87647-2-git-send-email-nic@opencloud.tech
State Not Applicable
Headers show

Commit Message

nickcooper-zhangtonghao March 3, 2017, 9:37 a.m. UTC
The ports which are attached mrouters or hosts, were destroyed
by users via ovs-vsctl commands. Currently the vswitch will
segfault if users use "ovs-appctl mdb/show" to show mdb info.
This patch avoids a segfault.

or

    ofputil_port_to_string(ofbundle_get_a_port(bundle)->up.ofp_port,

Signed-off-by: nickcooper-zhangtonghao <nic@opencloud.tech>
---
 ofproto/ofproto-dpif.c | 1 +
 1 file changed, 1 insertion(+)

Comments

Ben Pfaff March 3, 2017, 7:34 p.m. UTC | #1
Thank you for the bug fixes.  I applied them to master, branch-2.7, and
branch-2.6.
diff mbox

Patch

diff --git a/ofproto/ofproto-dpif.c b/ofproto/ofproto-dpif.c
index 366b7a2..1e1b107 100644
--- a/ofproto/ofproto-dpif.c
+++ b/ofproto/ofproto-dpif.c
@@ -2756,6 +2756,7 @@  bundle_destroy(struct ofbundle *bundle)
     }
 
     bundle_flush_macs(bundle, true);
+    mcast_snooping_flush_bundle(ofproto->ms, bundle);
     hmap_remove(&ofproto->bundles, &bundle->hmap_node);
     free(bundle->name);
     free(bundle->trunks);