I think I posted the same patch yesterday. On Feb 16, 2013 12:32 PM, "Ethan Jackson" <et...@nicira.com> wrote:
> The existing code had gotten the endianness wrong. Found by > sparse. > > Signed-off-by: Ethan Jackson <et...@nicira.com> > --- > 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 > dev@openvswitch.org > http://openvswitch.org/mailman/listinfo/dev >
_______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev