From: Gurucharan Shetty <gshetty@nicira.com>
Date: Mon, 28 Sep 2015 19:41:46 +0000 (-0700)
Subject: ovn-controller: Fix a case wherein tag can be zero.
X-Git-Tag: v2.12.3~6715
X-Git-Url: https://git.proxmox.com/?a=commitdiff_plain;h=3069cbeb1087781d886f905d35867ab9a80d41c9;p=mirror_ovs.git

ovn-controller: Fix a case wherein tag can be zero.

If the ovn-nb DB's logical_port table is populated such
that tag is zero, we should not consider that record.

Signed-off-by: Gurucharan Shetty <gshetty@nicira.com>
Acked-by: Russell Bryant <rbryant@redhat.com>
---

diff --git a/ovn/controller/physical.c b/ovn/controller/physical.c
index f57acbc05..528091602 100644
--- a/ovn/controller/physical.c
+++ b/ovn/controller/physical.c
@@ -253,9 +253,12 @@ physical_run(struct controller_ctx *ctx, enum mf_field_id mff_ovn_geneve,
             }
             ofport = u16_to_ofp(simap_get(&localnet_to_ofport, network));
         } else if (binding->parent_port) {
+            if (!binding->tag || !*binding->tag) {
+                continue;
+            }
             ofport = u16_to_ofp(simap_get(&localvif_to_ofport,
                                           binding->parent_port));
-            if (ofport && binding->tag) {
+            if (ofport) {
                 tag = *binding->tag;
             }
         } else {