[ovs-dev] [PATCH] nicira-ext: Add size check for struct nx_role_request.

Ethan Jackson ethan at nicira.com
Thu Feb 2 01:57:49 UTC 2012


Looks good,

Ethan

On Thu, Jan 26, 2012 at 15:50, Ben Pfaff <blp at nicira.com> wrote:
> We try to have such a size check every protocol structure.
>
> Signed-off-by: Ben Pfaff <blp at nicira.com>
> ---
>  include/openflow/nicira-ext.h |    1 +
>  1 files changed, 1 insertions(+), 0 deletions(-)
>
> diff --git a/include/openflow/nicira-ext.h b/include/openflow/nicira-ext.h
> index 6921803..62180a7 100644
> --- a/include/openflow/nicira-ext.h
> +++ b/include/openflow/nicira-ext.h
> @@ -276,6 +276,7 @@ struct nx_role_request {
>     struct nicira_header nxh;
>     ovs_be32 role;              /* One of NX_ROLE_*. */
>  };
> +OFP_ASSERT(sizeof(struct nx_role_request) == 20);
>
>  enum nx_role {
>     NX_ROLE_OTHER,              /* Default role, full access. */
> --
> 1.7.2.5
>
> _______________________________________________
> dev mailing list
> dev at openvswitch.org
> http://openvswitch.org/mailman/listinfo/dev



More information about the dev mailing list