When testing megaflow bond implement I ran into a case where
dpif->handler[0].channels is NULL. Indexing into channels causes
vswitchd to crash. Not sure this is the best way to fix it, but
it worked for me.

Signed-off-by: Andy Zhou <az...@nicira.com>
---
 lib/dpif-linux.c |    4 ++++
 1 file changed, 4 insertions(+)

diff --git a/lib/dpif-linux.c b/lib/dpif-linux.c
index a575b78..9ba6e6a 100644
--- a/lib/dpif-linux.c
+++ b/lib/dpif-linux.c
@@ -441,6 +441,10 @@ vport_del_channels(struct dpif_linux *dpif, odp_port_t 
port_no)
         return;
     }
 
+    if (!dpif->handlers[0].channels) {
+        return;
+    }
+
     /* Since the sock can only be assigned in either all or none
      * of "dpif->handlers" channels, the following check would
      * suffice. */
-- 
1.7.9.5

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

Reply via email to