Skip to contentSkip to conversation

ACPI / tables: test the correct variable

PATCH Applied Dan Carpenter 2 participants 2 messages Sep 22, 2015 – Sep 24, 2015

Subsystems (2)
Kernel janitors ACPI
lore
Share & tools
lore

Dan Carpenter

author
#0 lore raw

PATCH 1/1 ACPI / tables: test the correct variable

The intent was to test "proc[i].handler" instead of "proc->handler".

Signed-off-by Dan Carpenter <[email protected]>
drivers/acpi/tables.c +2 -1
index c1ff58d..6c0f079 100644
@@ -268,7 +268,8 @@ acpi_parse_entries_array(char *id, unsigned long table_size,
 		for (i = 0; i < proc_num; i++) {
 			if (entry->type != proc[i].id)
 				continue;
-			if (!proc->handler || proc[i].handler(entry, table_end))
+			if (!proc[i].handler ||
+			     proc[i].handler(entry, table_end))
 				return -EINVAL;
 
 			proc->count++;
Applied
#1 lore raw
Quoted 19 lines · Dan Carpenter

The intent was to test "proc[i].handler" instead of "proc->handler".

Signed-off-by Dan Carpenter <[email protected]>
drivers/acpi/tables.c +2 -2
index c1ff58d..6c0f079 100644
@@ -268,7 +268,8 @@ acpi_parse_entries_array(char *id, unsigned long table_size,
 		for (i = 0; i < proc_num; i++) {
 			if (entry->type != proc[i].id)
 				continue;
-			if (!proc->handler || proc[i].handler(entry, table_end))
+			if (!proc[i].handler ||
+			     proc[i].handler(entry, table_end))
 				return -EINVAL;
 
 			proc->count++;
--
Quoted 3 lines

To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html

Applied, thanks!