Re: [PATCH net-next 03/13] sctp: remove an if() that is always true

2018-04-27 Thread Neil Horman
On Fri, Apr 27, 2018 at 03:13:49PM -0300, Marcelo Ricardo Leitner wrote: > On Fri, Apr 27, 2018 at 06:50:50AM -0400, Neil Horman wrote: > > On Thu, Apr 26, 2018 at 04:58:52PM -0300, Marcelo Ricardo Leitner wrote: > > > As noticed by Xin Long, the if() here is always true as PMTU can never > > > be

Re: [PATCH net-next 03/13] sctp: remove an if() that is always true

2018-04-27 Thread Marcelo Ricardo Leitner
On Fri, Apr 27, 2018 at 06:50:50AM -0400, Neil Horman wrote: > On Thu, Apr 26, 2018 at 04:58:52PM -0300, Marcelo Ricardo Leitner wrote: > > As noticed by Xin Long, the if() here is always true as PMTU can never > > be 0. > > > > Reported-by: Xin Long > > Signed-off-by: Marcelo Ricardo Leitner > >

Re: [PATCH net-next 03/13] sctp: remove an if() that is always true

2018-04-27 Thread Neil Horman
On Thu, Apr 26, 2018 at 04:58:52PM -0300, Marcelo Ricardo Leitner wrote: > As noticed by Xin Long, the if() here is always true as PMTU can never > be 0. > > Reported-by: Xin Long > Signed-off-by: Marcelo Ricardo Leitner > --- > net/sctp/associola.c | 6 ++ > 1 file changed, 2 insertions(+)

[PATCH net-next 03/13] sctp: remove an if() that is always true

2018-04-26 Thread Marcelo Ricardo Leitner
As noticed by Xin Long, the if() here is always true as PMTU can never be 0. Reported-by: Xin Long Signed-off-by: Marcelo Ricardo Leitner --- net/sctp/associola.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/net/sctp/associola.c b/net/sctp/associola.c index b3aa952