]> git.ipfire.org Git - thirdparty/kernel/linux.git/commit
sctp: validate cached peer INIT chunk length in COOKIE_ECHO processing
authorXin Long <lucien.xin@gmail.com>
Tue, 2 Jun 2026 01:06:06 +0000 (21:06 -0400)
committerJakub Kicinski <kuba@kernel.org>
Thu, 4 Jun 2026 15:30:07 +0000 (08:30 -0700)
commit0861615c28de668669d748ef4eb913ea9262d13b
treeb165fa098021ca0474b5d7d0ee46db43ef2bad14
parent899ee91156e57784090c5565e4f31bd7dbffbc5a
sctp: validate cached peer INIT chunk length in COOKIE_ECHO processing

When a listening SCTP server processes a COOKIE_ECHO chunk, the cached
peer INIT chunk embedded after the cookie is parsed and its parameters
are later walked by sctp_process_init() using sctp_walk_params().

However, the chunk header length of this cached INIT chunk was not
validated against the remaining buffer in the COOKIE_ECHO payload. If
the length field is inflated, the parameter walk can run beyond the
actual received data, leading to out-of-bounds reads and potential
memory corruption during later parameter handling (e.g. STATE_COOKIE
processing and kmemdup() copies).

Add a bounds check in sctp_unpack_cookie() to ensure the cached INIT
chunk length does not exceed the available data in the COOKIE_ECHO
buffer before it is used.

Fixes: 1da177e4c3f4 ("Linux-2.6.12-rc2")
Reported-by: Brian Geffon <bgeffon@google.com>
Signed-off-by: Xin Long <lucien.xin@gmail.com>
Link: https://patch.msgid.link/eb60825fa22d6f9e663c7d4dbb69f397b5d34d42.1780362366.git.lucien.xin@gmail.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
net/sctp/sm_make_chunk.c