Re: [PATCH] platform/x86: toshiba_acpi: validate ACPI object type in tci_raw()
From: Tristan Madani
Date: Tue Nov 10 2026 - 18:48:29 EST
On Mon, 9 Nov 2026, Ilpo Järvinen wrote:
> Your ACPI validation patches seem to lack information whether these
> actually have been observed to occur on some device.
No, this was found by code audit, not observed on hardware. The
tci_raw() function accesses the returned ACPI object as a package
without checking the type field first. If the firmware method ever
returns something other than a package (e.g. an integer or buffer),
the code interprets whatever the union holds as package.count and
package.elements, which is undefined.
The sibling hp-bioscfg driver already validates the return type in
the same way (biosattr-interface.c line 159), but toshiba_acpi
predates that and never picked up the check.
I can resend as v2 with the changelog explicitly noting this is a
hardening fix found by code inspection, if that helps.