[ovs-dev] [PATCH 1/2] dpif-linux: Set vxlan destination port properly.

Ben Pfaff blp at nicira.com
Sat Feb 16 21:01:09 UTC 2013


I think I posted the same patch yesterday.
On Feb 16, 2013 12:32 PM, "Ethan Jackson" <ethan at nicira.com> wrote:

> The existing code had gotten the endianness wrong.  Found by
> sparse.
>
> Signed-off-by: Ethan Jackson <ethan at 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 at openvswitch.org
> http://openvswitch.org/mailman/listinfo/dev
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openvswitch.org/pipermail/ovs-dev/attachments/20130216/6bb3cb81/attachment-0003.html>


More information about the dev mailing list