On Fri, Feb 15, 2013 at 11:24 AM, Ben Pfaff <[email protected]> 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 <[email protected]>

Looks good to me.
_______________________________________________
dev mailing list
[email protected]
http://openvswitch.org/mailman/listinfo/dev

Reply via email to