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