mirror of
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
synced 2026-08-31 09:20:13 -04:00
drm/amdkfd: Add bounds check for CRAT subtype length
The CRAT parser validates that the subtype header fits within the image,
but does not verify that the advertised subtype length fits. A malformed
CRAT table with an oversized length field causes out-of-bounds reads when
kfd_parse_subtype() casts the header to specific subtype structures.
Add validation that sub_type_hdr + length does not exceed the image
boundary before parsing the subtype contents.
Signed-off-by: William Palacek <William.Palacek@amd.com>
Reviewed-by: Alysa Liu <Alysa.Liu@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
(cherry picked from commit 48e1d1e6e8)
Cc: stable@vger.kernel.org
This commit is contained in:
committed by
Alex Deucher
parent
35699ae99d
commit
6e7566ba47
@@ -1412,6 +1412,15 @@ int kfd_parse_crat_table(void *crat_image, struct list_head *device_list,
|
||||
break;
|
||||
}
|
||||
|
||||
/* Validate subtype fits within remaining image */
|
||||
if ((char *)sub_type_hdr + sub_type_hdr->length >
|
||||
(char *)crat_image + image_len) {
|
||||
pr_warn("CRAT subtype length %u exceeds image bounds\n",
|
||||
sub_type_hdr->length);
|
||||
ret = -EINVAL;
|
||||
break;
|
||||
}
|
||||
|
||||
if (sub_type_hdr->flags & CRAT_SUBTYPE_FLAGS_ENABLED) {
|
||||
ret = kfd_parse_subtype(sub_type_hdr, device_list);
|
||||
if (ret)
|
||||
|
||||
Reference in New Issue
Block a user