diff options
| author | Marcelo Ricardo Leitner <marcelo.leitner@gmail.com> | 2018-04-26 21:58:52 +0200 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2018-04-27 20:35:22 +0200 |
| commit | c88da20f95ad0bfa49e3e9035c26aaac0b05d11d (patch) | |
| tree | 40df20287e49af647cff4b2dcef7158459607e8e | |
| parent | sctp: move transport pathmtu calc away of sctp_assoc_add_peer (diff) | |
| download | linux-c88da20f95ad0bfa49e3e9035c26aaac0b05d11d.tar.xz linux-c88da20f95ad0bfa49e3e9035c26aaac0b05d11d.zip | |
sctp: remove an if() that is always true
As noticed by Xin Long, the if() here is always true as PMTU can never
be 0.
Reported-by: Xin Long <lucien.xin@gmail.com>
Signed-off-by: Marcelo Ricardo Leitner <marcelo.leitner@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
| -rw-r--r-- | net/sctp/associola.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/net/sctp/associola.c b/net/sctp/associola.c index b3aa95222bd5..c5ed09cfa842 100644 --- a/net/sctp/associola.c +++ b/net/sctp/associola.c @@ -1397,10 +1397,8 @@ void sctp_assoc_sync_pmtu(struct sctp_association *asoc) pmtu = t->pathmtu; } - if (pmtu) { - asoc->pathmtu = pmtu; - asoc->frag_point = sctp_frag_point(asoc, pmtu); - } + asoc->pathmtu = pmtu; + asoc->frag_point = sctp_frag_point(asoc, pmtu); pr_debug("%s: asoc:%p, pmtu:%d, frag_point:%d\n", __func__, asoc, asoc->pathmtu, asoc->frag_point); |