[PATCH] acpi: LPIT: check for monotonically increasing UID
Alex Hung
alex.hung at canonical.com
Tue Nov 21 03:21:57 UTC 2017
Signed-off-by: Alex Hung <alex.hung at canonical.com>
---
src/acpi/lpit/lpit.c | 12 ++++++++++++
1 file changed, 12 insertions(+)
diff --git a/src/acpi/lpit/lpit.c b/src/acpi/lpit/lpit.c
index eb4e3af..de7073c 100644
--- a/src/acpi/lpit/lpit.c
+++ b/src/acpi/lpit/lpit.c
@@ -162,8 +162,10 @@ static int lpit_test1(fwts_framework *fw)
/* Got enough data to be able to inspect the initial 2 x 32 bit words.. */
while (length > 8) {
+ fwts_acpi_table_lpit_c_state *lpi = (fwts_acpi_table_lpit_c_state *)data;
uint32_t *ptr = (uint32_t *)data;
uint32_t lpi_length = *(ptr + 1);
+ uint16_t uid = 0;
/* Stated LPI length must not be longer than what's left in the table */
if (length < lpi_length) {
@@ -181,6 +183,16 @@ static int lpit_test1(fwts_framework *fw)
switch (*ptr) {
case 0x0:
lpit_check_type_0(fw, &length, &data, &passed);
+
+ if (uid > lpi->id && (lpi->id - uid) > 1) {
+ passed = false;
+ fwts_failed(fw, LOG_LEVEL_CRITICAL,
+ "LPITNativeCStateBadUID",
+ "Unique ID of Native C-state based LPI "
+ "structure must be a zero-based monotonically "
+ "increasing value");
+ }
+ uid = lpi->id;
break;
default:
passed = false;
--
2.7.4
More information about the fwts-devel
mailing list