[ovs-dev] [PATCH] dpif-linux: Fix byte-swapping direction in nl_msg_put_u16() call.

Jesse Gross jesse at nicira.com
Fri Feb 15 21:14:52 UTC 2013


On Fri, Feb 15, 2013 at 11:24 AM, Ben Pfaff <blp at nicira.com> wrote:
> OVS_TUNNEL_ATTR_DST_PORT expects a u16, tnl_cfg->dst_port is a be16, so
> we want ntohs() instead of htons().
>
> In practice htons() and ntohs() perform the same operation, so this does
> not fix a real bug.
>
> Found by sparse.
>
> Signed-off-by: Ben Pfaff <blp at nicira.com>

Looks good to me.



More information about the dev mailing list