aboutsummaryrefslogtreecommitdiffstats
path: root/net/openvswitch
diff options
context:
space:
mode:
authorPravin B Shelar <pshelar@nicira.com>2014-12-01 02:04:17 -0500
committerDavid S. Miller <davem@davemloft.net>2014-12-06 00:42:16 -0500
commitf2a01517f2a1040a0b156f171a7cefd748f2fd03 (patch)
tree6de3f5686f19c2a46a0d6e9fa653e662f5d19841 /net/openvswitch
parent4fc11e61f46dd66730a18e8136607386007ee8c9 (diff)
openvswitch: Fix flow mask validation.
Following patch fixes typo in the flow validation. This prevented installation of ARP and IPv6 flows. Fixes: 19e7a3df72 ("openvswitch: Fix NDP flow mask validation") Signed-off-by: Pravin B Shelar <pshelar@nicira.com> Reviewed-by: Thomas Graf <tgraf@suug.ch> Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/openvswitch')
-rw-r--r--net/openvswitch/flow_netlink.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/net/openvswitch/flow_netlink.c b/net/openvswitch/flow_netlink.c
index 089b195c064a..918e96645b05 100644
--- a/net/openvswitch/flow_netlink.c
+++ b/net/openvswitch/flow_netlink.c
@@ -145,7 +145,7 @@ static bool match_validate(const struct sw_flow_match *match,
145 if (match->key->eth.type == htons(ETH_P_ARP) 145 if (match->key->eth.type == htons(ETH_P_ARP)
146 || match->key->eth.type == htons(ETH_P_RARP)) { 146 || match->key->eth.type == htons(ETH_P_RARP)) {
147 key_expected |= 1 << OVS_KEY_ATTR_ARP; 147 key_expected |= 1 << OVS_KEY_ATTR_ARP;
148 if (match->mask && (match->mask->key.tp.src == htons(0xff))) 148 if (match->mask && (match->mask->key.eth.type == htons(0xffff)))
149 mask_allowed |= 1 << OVS_KEY_ATTR_ARP; 149 mask_allowed |= 1 << OVS_KEY_ATTR_ARP;
150 } 150 }
151 151
@@ -220,7 +220,7 @@ static bool match_validate(const struct sw_flow_match *match,
220 htons(NDISC_NEIGHBOUR_SOLICITATION) || 220 htons(NDISC_NEIGHBOUR_SOLICITATION) ||
221 match->key->tp.src == htons(NDISC_NEIGHBOUR_ADVERTISEMENT)) { 221 match->key->tp.src == htons(NDISC_NEIGHBOUR_ADVERTISEMENT)) {
222 key_expected |= 1 << OVS_KEY_ATTR_ND; 222 key_expected |= 1 << OVS_KEY_ATTR_ND;
223 if (match->mask && (match->mask->key.tp.src == htons(0xffff))) 223 if (match->mask && (match->mask->key.tp.src == htons(0xff)))
224 mask_allowed |= 1 << OVS_KEY_ATTR_ND; 224 mask_allowed |= 1 << OVS_KEY_ATTR_ND;
225 } 225 }
226 } 226 }