[ovs-dev] [tests+nxm-ofctl 23/42] ofp-print: Fix handling of zero-length actions.
Ethan Jackson
ethan at nicira.com
Thu Dec 2 22:27:19 UTC 2010
Looks God.
On Tue, Nov 23, 2010 at 2:43 PM, Ben Pfaff <blp at nicira.com> wrote:
> Without this change, zero-length ofp_actions cause an infinite loop.
> ---
> lib/ofp-print.c | 5 +++++
> 1 files changed, 5 insertions(+), 0 deletions(-)
>
> diff --git a/lib/ofp-print.c b/lib/ofp-print.c
> index e3cae78..97fd065 100644
> --- a/lib/ofp-print.c
> +++ b/lib/ofp-print.c
> @@ -314,6 +314,11 @@ ofp_print_action(struct ds *string, const struct ofp_action_header *ah,
> return -1;
> }
>
> + if (!len) {
> + ds_put_format(string, "***zero-length action***\n");
> + return 8;
> + }
> +
> if ((len % OFP_ACTION_ALIGN) != 0) {
> ds_put_format(string,
> "***action %"PRIu16" length not a multiple of %d***\n",
> --
> 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