[ovs-dev] [PATCH] ovn: Fix a northd bug.

Alex Wang alexw at nicira.com
Tue May 19 01:58:15 UTC 2015


The 'chassis' member in the 'struct sbrec_binding' must always be
non-null.  However, this is not case when creating the binding
in "set_bindings()".  And it causes segfault while starting northd
with existing ovnnb configuration.

This commit fixes the bug by always setting the 'chassis' to an
empty string.

Signed-off-by: Alex Wang <alexw at nicira.com>
---
 ovn/northd/ovn-northd.c |    1 +
 1 file changed, 1 insertion(+)

diff --git a/ovn/northd/ovn-northd.c b/ovn/northd/ovn-northd.c
index cfad6be..026216b 100644
--- a/ovn/northd/ovn-northd.c
+++ b/ovn/northd/ovn-northd.c
@@ -578,6 +578,7 @@ set_bindings(struct northd_context *ctx)
                 sbrec_binding_set_tag(binding, lport->tag, lport->n_tag);
             }
 
+            sbrec_binding_set_chassis(binding, "");
             sbrec_binding_set_tunnel_key(binding, tunnel_key);
             sbrec_binding_set_logical_datapath(binding, logical_datapath);
 
-- 
1.7.9.5




More information about the dev mailing list