[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