On 12/02/2014 05:31 PM, Jarkko Sakkinen wrote:

+struct acpi_tpm2 {
+       struct acpi_table_header hdr;
+       u16 platform_class;
+       u16 reserved;
+       u64 control_area_pa;
+       u32 start_method;
+} __packed;
[...]
+       }
+
+       if (buf->hdr.length != sizeof(struct acpi_tpm2)) {

This should be

if (buf->hdr.length < sizeof(struct acpi_tpm2)) {

since the ACPI TPM2 table may have a variable number of 'Platform Specific Parameters' beyond the 'Start Method Field'. What we don't want to have is less, but more is fine.

   Stefan

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to