diff options
| author | Marcelo Ricardo Leitner <marcelo.leitner@gmail.com> | 2021-07-27 23:40:54 -0300 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2021-08-04 12:23:46 +0200 |
| commit | d9b6f0a200bef5fb9ba00e4e7d0d51f0b0fa5c6f (patch) | |
| tree | 91aba7ed42764a5a97cc0ad7657caca594a964fc | |
| parent | 2a7b6a5203b6f80c35fcd66c995201cb54f4178f (diff) | |
| download | linux-stable-d9b6f0a200bef5fb9ba00e4e7d0d51f0b0fa5c6f.tar.gz linux-stable-d9b6f0a200bef5fb9ba00e4e7d0d51f0b0fa5c6f.tar.bz2 linux-stable-d9b6f0a200bef5fb9ba00e4e7d0d51f0b0fa5c6f.zip | |
sctp: fix return value check in __sctp_rcv_asconf_lookup
[ Upstream commit 557fb5862c9272ad9b21407afe1da8acfd9b53eb ]
As Ben Hutchings noticed, this check should have been inverted: the call
returns true in case of success.
Reported-by: Ben Hutchings <ben@decadent.org.uk>
Fixes: 0c5dc070ff3d ("sctp: validate from_addr_param return")
Signed-off-by: Marcelo Ricardo Leitner <marcelo.leitner@gmail.com>
Reviewed-by: Xin Long <lucien.xin@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Signed-off-by: Sasha Levin <sashal@kernel.org>
| -rw-r--r-- | net/sctp/input.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/sctp/input.c b/net/sctp/input.c index 203aaefd8d04..23c4f14bec25 100644 --- a/net/sctp/input.c +++ b/net/sctp/input.c @@ -1132,7 +1132,7 @@ static struct sctp_association *__sctp_rcv_asconf_lookup( if (unlikely(!af)) return NULL; - if (af->from_addr_param(&paddr, param, peer_port, 0)) + if (!af->from_addr_param(&paddr, param, peer_port, 0)) return NULL; return __sctp_lookup_association(net, laddr, &paddr, transportp); |