The code dereferences "chassis", which could be null if chassis_run()
returns null. "chassis" will always be null if "chassis_id" is null, so
checking "chassis" is sufficient to check both.
Found by inspection.
Signed-off-by: Justin Pettit <jpet...@ovn.org>
---
ovn/controller/ovn-controller.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/ovn/controller/ovn-controller.c b/ovn/controller/ovn-controller.c
index 5f2f90a..49821f7 100644
--- a/ovn/controller/ovn-controller.c
+++ b/ovn/controller/ovn-controller.c
@@ -427,7 +427,7 @@ main(int argc, char *argv[])
&all_lports);
}
- if (br_int && chassis_id) {
+ if (br_int && chassis) {
patch_run(&ctx, br_int, chassis_id, &local_datapaths,
&patched_datapaths);
--
1.9.1
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
