From mboxrd@z Thu Jan 1 00:00:00 1970 From: Vlad Yasevich Subject: Re: [PATCH] sctp: IPsec rules are ineffective with ipv6 Date: Thu, 28 Jan 2010 10:24:18 -0500 Message-ID: <4B61AC22.2050907@hp.com> References: <4B6049EB.8030803@dev.6wind.com> <20100128.055148.127214200.davem@davemloft.net> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Cc: nicolas.dichtel@dev.6wind.com, netdev@vger.kernel.org, linux-sctp@vger.kernel.org To: David Miller Return-path: Received: from g4t0017.houston.hp.com ([15.201.24.20]:21097 "EHLO g4t0017.houston.hp.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755036Ab0A1PYW (ORCPT ); Thu, 28 Jan 2010 10:24:22 -0500 In-Reply-To: <20100128.055148.127214200.davem@davemloft.net> Sender: netdev-owner@vger.kernel.org List-ID: David Miller wrote: > From: Nicolas Dichtel > Date: Wed, 27 Jan 2010 15:12:59 +0100 > >> xfrm_lookup() is missing in sctp_v6_xmit(), add it. >> >> Signed-off-by: Junwei Zhang >> Signed-off-by: Nicolas Dichtel > > Doing this every transmit packet is overkill. > > Whatever calculates the route that ends up in skb_dst(skb) > should be making this xfrm_lookup() call, not here. > Hmm.. Interesting. Looks like ip_route_output_key() will do xfrm_lookup for you, but there is no ipv6 route lookup call that will do the same thing. I guess we'll need to add an xfrm_lookup call in sctp_v6_get_dst(). -vlad