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:22:17 +0200 |
| commit | 1ad35d7d7fb3d160535b301ba0763e7ea373b062 (patch) | |
| tree | 618d5df42d702e1282cc09e7a68aa5483d15bf07 | |
| parent | 766807b77ba14c5dfde8dd1823e637ad9a54b88a (diff) | |
| download | linux-stable-1ad35d7d7fb3d160535b301ba0763e7ea373b062.tar.gz linux-stable-1ad35d7d7fb3d160535b301ba0763e7ea373b062.tar.bz2 linux-stable-1ad35d7d7fb3d160535b301ba0763e7ea373b062.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 1af35b69e99e..90428c59cfaf 100644 --- a/net/sctp/input.c +++ b/net/sctp/input.c @@ -1125,7 +1125,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); |