Good catch, Han.

Acked-by: Mark Michelson <[email protected]>

On 11/3/23 01:40, Han Zhou wrote:
It is not causing any problem so far, but fix it to avoid potential
problem in the future.

Fixes: c6b20c9940ed ("ovn-ic: Support IGMP/MLD in multi-AZ deployments.")
Signed-off-by: Han Zhou <[email protected]>
---
  controller/physical.c | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/controller/physical.c b/controller/physical.c
index 72e88a203a66..41c38abc4a58 100644
--- a/controller/physical.c
+++ b/controller/physical.c
@@ -2043,7 +2043,7 @@ consider_mc_group(struct ovsdb_idl_index 
*sbrec_port_binding_by_name,
                  local_output_pb(port->tunnel_key, &remote_ofpacts);
                  local_output_pb(port->tunnel_key, &remote_ofpacts_ramp);
              }
-        } if (!strcmp(port->type, "remote")) {
+        } else if (!strcmp(port->type, "remote")) {
              if (port->chassis) {
                  put_load(port->tunnel_key, MFF_LOG_OUTPORT, 0, 32,
                           &remote_ofpacts);

_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to