I think I posted the same patch yesterday. On Feb 16, 2013 12:32 PM, "Ethan Jackson" <[email protected]> wrote:
> The existing code had gotten the endianness wrong. Found by > sparse. > > Signed-off-by: Ethan Jackson <[email protected]> > --- > lib/dpif-linux.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/lib/dpif-linux.c b/lib/dpif-linux.c > index 3cf1225..aa1739d 100644 > --- a/lib/dpif-linux.c > +++ b/lib/dpif-linux.c > @@ -521,7 +521,7 @@ dpif_linux_port_add(struct dpif *dpif_, struct netdev > *netdev, > if (tnl_cfg && tnl_cfg->dst_port != 0) { > ofpbuf_use_stack(&options, options_stub, sizeof options_stub); > nl_msg_put_u16(&options, OVS_TUNNEL_ATTR_DST_PORT, > - htons(tnl_cfg->dst_port)); > + ntohs(tnl_cfg->dst_port)); > request.options = options.data; > request.options_len = options.size; > } > -- > 1.7.9.5 > > _______________________________________________ > dev mailing list > [email protected] > http://openvswitch.org/mailman/listinfo/dev >
_______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
