netdev - Re: [PATCH net v3] openvswitch: Fix egress tunnel info.

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:	Sat, 24 Oct 2015 12:26:04 +0200
From:	Jiri Benc <jbenc@...hat.com>
To:	Pravin Shelar <pshelar@...ira.com>
Cc:	netdev <netdev@...r.kernel.org>
Subject: Re: [PATCH net v3] openvswitch: Fix egress tunnel info.

On Fri, 23 Oct 2015 12:47:09 -0700, Pravin Shelar wrote:
> This is not complete code. I found couple of issues with it.

That's likely, I didn't test it :-)

> This code does not copy lwtunnel_state state into new dst.

Not sure what state you mean. All the relevant state should be in
struct ip_tunnel_info.

> And it is converting lwtunnel dst into metadata dst.

I did that intentionally. But your comment made me realize that I was
too much focused on the tunnel implementation side. This function,
should it be used outside of ovs, could be called earlier than the skb
is handed to the tunnel. Then we of course need a valid dst->output and
this approach (converting lwtunnel data to metadata_dst) would fail.
This is the same problem we have with IPv6 ndisc replies.

This needs more thinking and is out of scope of what you're trying to
solve. Sorry for the noise. But at least it made me realize that the
ndisc problem (that I still don't have a good solution for) is broader.

Thanks,

 Jiri

-- 
Jiri Benc
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html