Avoid null pointer dereference in fdb_calculate_active_tunnels()
when integration bridge isn't present. This is easily encountered
by executing "make sandbox SANDBOXFLAGS=--ovn".

Fixes: 3475695ea61c ("ovn: l3ha, enable bfd between tunnel endpoints")
Signed-off-by: Lance Richardson <lrich...@redhat.com>
---
 ovn/controller/bfd.c | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/ovn/controller/bfd.c b/ovn/controller/bfd.c
index a02562338..03cf377e1 100644
--- a/ovn/controller/bfd.c
+++ b/ovn/controller/bfd.c
@@ -62,6 +62,11 @@ bfd_calculate_active_tunnels(const struct ovsrec_bridge 
*br_int,
 {
     int i;
 
+    if (!br_int) {
+        /* Nothing to do if integration bridge doesn't exist. */
+        return;
+    }
+
     for (i = 0; i < br_int->n_ports; i++) {
         const struct ovsrec_port *port_rec = br_int->ports[i];
 
-- 
2.13.3

_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to