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

Reply via email to