On Fri, Feb 15, 2013 at 01:14:52PM -0800, Jesse Gross wrote:
> 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.

Thanks, applied to master.
_______________________________________________
dev mailing list
[email protected]
http://openvswitch.org/mailman/listinfo/dev

Reply via email to