mirror of
git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
synced 2025-08-05 16:54:27 +00:00
ACPI PPTT: Fix coding mistakes in a couple of sizeof() calls
The end of table checks should be done with the structure size, but 2 of the 3 similar calls use the pointer size. Signed-off-by: Jean-Marc Eurin <jmeurin@google.com> Link: https://patch.msgid.link/20250402001542.2600671-1-jmeurin@google.com [ rjw: Subject edits ] Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
This commit is contained in:
parent
0af2f6be1b
commit
7ab4f0e37a
1 changed files with 2 additions and 2 deletions
|
@ -229,7 +229,7 @@ static int acpi_pptt_leaf_node(struct acpi_table_header *table_hdr,
|
||||||
node_entry = ACPI_PTR_DIFF(node, table_hdr);
|
node_entry = ACPI_PTR_DIFF(node, table_hdr);
|
||||||
entry = ACPI_ADD_PTR(struct acpi_subtable_header, table_hdr,
|
entry = ACPI_ADD_PTR(struct acpi_subtable_header, table_hdr,
|
||||||
sizeof(struct acpi_table_pptt));
|
sizeof(struct acpi_table_pptt));
|
||||||
proc_sz = sizeof(struct acpi_pptt_processor *);
|
proc_sz = sizeof(struct acpi_pptt_processor);
|
||||||
|
|
||||||
while ((unsigned long)entry + proc_sz < table_end) {
|
while ((unsigned long)entry + proc_sz < table_end) {
|
||||||
cpu_node = (struct acpi_pptt_processor *)entry;
|
cpu_node = (struct acpi_pptt_processor *)entry;
|
||||||
|
@ -270,7 +270,7 @@ static struct acpi_pptt_processor *acpi_find_processor_node(struct acpi_table_he
|
||||||
table_end = (unsigned long)table_hdr + table_hdr->length;
|
table_end = (unsigned long)table_hdr + table_hdr->length;
|
||||||
entry = ACPI_ADD_PTR(struct acpi_subtable_header, table_hdr,
|
entry = ACPI_ADD_PTR(struct acpi_subtable_header, table_hdr,
|
||||||
sizeof(struct acpi_table_pptt));
|
sizeof(struct acpi_table_pptt));
|
||||||
proc_sz = sizeof(struct acpi_pptt_processor *);
|
proc_sz = sizeof(struct acpi_pptt_processor);
|
||||||
|
|
||||||
/* find the processor structure associated with this cpuid */
|
/* find the processor structure associated with this cpuid */
|
||||||
while ((unsigned long)entry + proc_sz < table_end) {
|
while ((unsigned long)entry + proc_sz < table_end) {
|
||||||
|
|
Loading…
Add table
Reference in a new issue