]> git.ipfire.org Git - thirdparty/kernel/linux.git/commit
thunderbolt: property: Reject u32 wrap in tb_property_entry_valid()
authorMichael Bommarito <michael.bommarito@gmail.com>
Sun, 10 May 2026 23:16:56 +0000 (19:16 -0400)
committerMika Westerberg <mika.westerberg@linux.intel.com>
Mon, 11 May 2026 09:31:48 +0000 (11:31 +0200)
commit01deda0152066c6c955f0619114ea6afa070aaec
treed1a0c1b825482dfc184a53f85a493ae5e30939f6
parent5d6919055dec134de3c40167a490f33c74c12581
thunderbolt: property: Reject u32 wrap in tb_property_entry_valid()

entry->value is u32 and entry->length is u16; the sum is performed in
u32 and wraps.  A malicious XDomain peer can pick
value = 0xffffff00, length = 0x100 so the sum 0x100000000 wraps to 0
and passes the > block_len check.  tb_property_parse() then passes
entry->value to parse_dwdata() as a dword offset into the property
block, reading attacker-directed memory far past the allocation.

For TEXT-typed entries with the "deviceid" or "vendorid" keys this
lands in xd->device_name / xd->vendor_name and is readable back via
the per-XDomain device_name / vendor_name sysfs attributes; the leak
is NUL-bounded (kstrdup() stops at the first zero byte) and
untargeted (the attacker picks a delta, not an absolute address).
DATA-typed entries are parsed into property->value.data but not
generically surfaced to userspace.

Use check_add_overflow() so a wrapped sum is rejected.

Fixes: cdae7c07e3e3 ("thunderbolt: Add support for XDomain properties")
Cc: stable@vger.kernel.org
Assisted-by: Claude:claude-opus-4-6
Assisted-by: Codex:gpt-5-4
Signed-off-by: Michael Bommarito <michael.bommarito@gmail.com>
Signed-off-by: Mika Westerberg <mika.westerberg@linux.intel.com>
drivers/thunderbolt/property.c