Commit 8199e4e9 authored by Wander Lairson Costa's avatar Wander Lairson Costa Committed by Zhengchao Shao
Browse files

netfilter: xt_sctp: validate the flag_info count 

stable inclusion
from stable-v5.10.195
commit 5541827d13cf19b905594eaee586527476efaa61
category: bugfix
bugzilla: https://gitee.com/src-openeuler/kernel/issues/I85CAQ
CVE: CVE-2023-39193

Reference: https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=5541827d13cf19b905594eaee586527476efaa61



--------------------------------

commit e9947649 upstream.

sctp_mt_check doesn't validate the flag_count field. An attacker can
take advantage of that to trigger a OOB read and leak memory
information.

Add the field validation in the checkentry function.

Fixes: 2e4e6a17 ("[NETFILTER] x_tables: Abstraction layer for {ip,ip6,arp}_tables")
Cc: stable@vger.kernel.org
Reported-by: default avatarLucas Leong <wmliang@infosec.exchange>
Signed-off-by: default avatarWander Lairson Costa <wander@redhat.com>
Signed-off-by: default avatarPablo Neira Ayuso <pablo@netfilter.org>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: default avatarZhengchao Shao <shaozhengchao@huawei.com>
parent 22cb1432

net/netfilter/xt_sctp.c

+2 −0
Original line number Diff line number Diff line
@@ -150,6 +150,8 @@ static int sctp_mt_check(const struct xt_mtchk_param *par) 
{

	const struct xt_sctp_info *info = par->matchinfo;



	if (info->flag_count > ARRAY_SIZE(info->flag_info))

		return -EINVAL;

	if (info->flags & ~XT_SCTP_VALID_FLAGS)

		return -EINVAL;

	if (info->invflags & ~XT_SCTP_VALID_FLAGS)