diff options
author | Pravin B Shelar <pshelar@nicira.com> | 2014-12-01 02:04:17 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2014-12-06 00:42:16 -0500 |
commit | f2a01517f2a1040a0b156f171a7cefd748f2fd03 (patch) | |
tree | 6de3f5686f19c2a46a0d6e9fa653e662f5d19841 /net/openvswitch | |
parent | 4fc11e61f46dd66730a18e8136607386007ee8c9 (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.c | 4 |
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 | } |