Signed-off-by: Ben Pfaff <[email protected]>
---
 lib/meta-flow.c | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/lib/meta-flow.c b/lib/meta-flow.c
index 1724e46e1b97..e3274b97f335 100644
--- a/lib/meta-flow.c
+++ b/lib/meta-flow.c
@@ -2287,6 +2287,12 @@ mf_set(const struct mf_field *mf,
         *err_str = NULL;
     }
 
+    /* The cases where 'mask' is all-1-bits or all-0-bits were already handled
+     * above[*], so the code below only needs to work for the remaining cases
+     * of a nontrivial mask.
+     *
+     * [*] Except where the field is a tunnel metadata field and 'mask' is
+     *     all-0-bits; see above. */
     switch (mf->id) {
     case MFF_CT_ZONE:
     case MFF_CT_NW_PROTO:
-- 
2.20.1

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

Reply via email to