Re: [PATCHv2 net] sctp: return back transport in __sctp_rcv_init_lookup

2016-10-31 Thread David Miller
From: marcelo.leit...@gmail.com Date: Mon, 31 Oct 2016 18:13:02 -0200 > On Mon, Oct 31, 2016 at 03:53:48PM -0400, David Miller wrote: >> From: Xin Long >> Date: Mon, 31 Oct 2016 00:42:35 +0800 >> >> > Prior to this patch, it used a local variable to save the transport that

Re: [PATCHv2 net] sctp: return back transport in __sctp_rcv_init_lookup

2016-10-31 Thread marcelo . leitner
On Mon, Oct 31, 2016 at 03:53:48PM -0400, David Miller wrote: > From: Xin Long > Date: Mon, 31 Oct 2016 00:42:35 +0800 > > > Prior to this patch, it used a local variable to save the transport that is > > looked up by __sctp_lookup_association(), and didn't return it back.

Re: [PATCHv2 net] sctp: return back transport in __sctp_rcv_init_lookup

2016-10-31 Thread David Miller
From: Xin Long Date: Mon, 31 Oct 2016 00:42:35 +0800 > Prior to this patch, it used a local variable to save the transport that is > looked up by __sctp_lookup_association(), and didn't return it back. But in > sctp_rcv, it is used to initialize chunk->transport. So when

Re: [PATCHv2 net] sctp: return back transport in __sctp_rcv_init_lookup

2016-10-31 Thread Xin Long
On Mon, Oct 31, 2016 at 6:51 PM, Marcelo Ricardo Leitner wrote: > On Mon, Oct 31, 2016 at 12:42:35AM +0800, Xin Long wrote: >> Prior to this patch, it used a local variable to save the transport that is >> looked up by __sctp_lookup_association(), and didn't return it

Re: [PATCHv2 net] sctp: return back transport in __sctp_rcv_init_lookup

2016-10-31 Thread Marcelo Ricardo Leitner
On Mon, Oct 31, 2016 at 12:42:35AM +0800, Xin Long wrote: > Prior to this patch, it used a local variable to save the transport that is > looked up by __sctp_lookup_association(), and didn't return it back. But in > sctp_rcv, it is used to initialize chunk->transport. So when hitting this, > even

[PATCHv2 net] sctp: return back transport in __sctp_rcv_init_lookup

2016-10-30 Thread Xin Long
Prior to this patch, it used a local variable to save the transport that is looked up by __sctp_lookup_association(), and didn't return it back. But in sctp_rcv, it is used to initialize chunk->transport. So when hitting this, even if it found the transport, it was still initializing