On Sat, Nov 12, 2011 at 1:01 PM, Ben Pfaff <b...@nicira.com> wrote: > diff --git a/datapath/flow.c b/datapath/flow.c > index 9786c37..6caca2a 100644 > --- a/datapath/flow.c > +++ b/datapath/flow.c > int flow_from_nlattrs(struct sw_flow_key *swkey, int *key_lenp, > const struct nlattr *attr) [...] > + if (attrs & (1 << OVS_KEY_ATTR_ETHERTYPE)) { > + swkey->eth.type = nla_get_be16(a[OVS_KEY_ATTR_ETHERTYPE]); > + if (ntohs(swkey->eth.type) < 1536) > + return -EINVAL; > + attrs &= ~(1 << OVS_KEY_ATTR_ETHERTYPE); > + } else { > + swkey->eth.type = htons(ETH_P_802_2); > + } > + > + > + if (swkey->eth.type == htons(ETH_P_IP)) {
Double blank line here but otherwise looks good. Acked-by: Jesse Gross <je...@nicira.com> _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev