[ovs-dev] [ofp-print 03/15] ofproto: Fix encoding of NXST_* replies.

Justin Pettit jpettit at nicira.com
Tue Dec 14 22:10:29 UTC 2010


Looks good.

--Justin


On Dec 14, 2010, at 12:23 PM, Ben Pfaff wrote:

> This only matter for NXST_AGGREGATE currently since NXST_FLOW has value 0.
> ---
> ofproto/ofproto.c |    2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/ofproto/ofproto.c b/ofproto/ofproto.c
> index 5cd76df..5fef092 100644
> --- a/ofproto/ofproto.c
> +++ b/ofproto/ofproto.c
> @@ -3258,7 +3258,7 @@ make_nxstats_reply(ovs_be32 xid, ovs_be32 subtype, size_t body_len)
>     nsm->type = htons(OFPST_VENDOR);
>     nsm->flags = htons(0);
>     nsm->vendor = htonl(NX_VENDOR_ID);
> -    nsm->subtype = htonl(subtype);
> +    nsm->subtype = subtype;
>     return msg;
> }
> 
> -- 
> 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