[ovs-dev] [ofp-print 01/18] ofp-print: Verify size of OFPAT_ENQUEUE actions.

Justin Pettit jpettit at nicira.com
Thu Dec 9 00:47:40 UTC 2010


Looks good.

--Justin


On Dec 8, 2010, at 4:26 PM, Ben Pfaff wrote:

> This fixes a bug, although not a very serious one.
> ---
> lib/ofp-print.c |    4 ++++
> 1 files changed, 4 insertions(+), 0 deletions(-)
> 
> diff --git a/lib/ofp-print.c b/lib/ofp-print.c
> index 4141b9f..40d096d 100644
> --- a/lib/ofp-print.c
> +++ b/lib/ofp-print.c
> @@ -298,6 +298,10 @@ ofp_print_action(struct ds *string, const struct ofp_action_header *ah,
>         [OFPAT_SET_TP_DST] = {
>             sizeof(struct ofp_action_tp_port),
>             sizeof(struct ofp_action_tp_port),
> +        },
> +        [OFPAT_ENQUEUE] = {
> +            sizeof(struct ofp_action_enqueue),
> +            sizeof(struct ofp_action_enqueue),
>         }
>         /* OFPAT_VENDOR is not here, since it would blow up the array size. */
>     };
> -- 
> 1.7.1
> 
> 
> _______________________________________________
> dev mailing list
> dev at openvswitch.org
> http://openvswitch.org/mailman/listinfo/dev_openvswitch.org





More information about the dev mailing list