On 04/25/2014 03:57 PM, Julian Anastasov wrote:
Hello, On Fri, 25 Apr 2014, Xufeng Zhang wrote:diff --git a/net/sctp/protocol.c b/net/sctp/protocol.c index c09757f..f8a5968 100644 --- a/net/sctp/protocol.c +++ b/net/sctp/protocol.c @@ -493,6 +493,10 @@ static void sctp_v4_get_dst(struct sctp_transport *t, union sctp_addr *saddr, (AF_INET == laddr->a.sa.sa_family)) { fl4->saddr = laddr->a.v4.sin_addr.s_addr; fl4->fl4_sport = laddr->a.v4.sin_port; + /* Reset flowi4_oif so that it won't be incorrectly used + * as input parameter to routing lookups. + */ + fl4->flowi4_oif = 0;May be something like this is more correct: flowi4_update_output(fl4, asoc->base.sk->sk_bound_dev_if, RT_CONN_FLAGS(asoc->base.sk), daddr->v4.sin_addr.s_addr, laddr->a.v4.sin_addr.s_addr);
Thanks! you are right, I should use the available API. I'll send V2 later. Thanks, Xufeng
rt = ip_route_output_key(sock_net(sk), fl4); if (!IS_ERR(rt)) { dst =&rt->dst;Regards -- Julian Anastasov<ja@xxxxxx>
-- To unsubscribe from this list: send the line "unsubscribe stable" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html