On Fri, Oct 21, 2011 at 5:12 PM, Ben Pfaff <b...@nicira.com> wrote: > diff --git a/datapath/datapath.c b/datapath/datapath.c > index de2f76b..d3147eb 100644 > --- a/datapath/datapath.c > +++ b/datapath/datapath.c > @@ -512,24 +512,29 @@ static int validate_actions(const struct nlattr *attr, > static int validate_sample(const struct nlattr *attr, > const struct sw_flow_key *key, int depth) [...] > - if (!a[OVS_SAMPLE_ATTR_PROBABILITY]) > + memset(attrs, 0, sizeof attrs);
Kernel style wants parentheses to go with sizeof but otherwise looks good to me. Acked-by: Jesse Gross <je...@nicira.com> _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev