diff options
| author | Len Brown <lenb@kernel.org> | 2009-03-15 23:36:38 -0400 |
|---|---|---|
| committer | Len Brown <len.brown@intel.com> | 2009-04-03 12:05:14 -0400 |
| commit | 53af9cfb37af5e03ee2b24c5d5c4963c34e5b765 (patch) | |
| tree | 337290e66795f10e785209e8891d4f4292ce5a89 /drivers | |
| parent | 2a9ef8e1a856be8e526bb9b10fb98c5012f6e3f8 (diff) | |
| download | op-kernel-dev-53af9cfb37af5e03ee2b24c5d5c4963c34e5b765.zip op-kernel-dev-53af9cfb37af5e03ee2b24c5d5c4963c34e5b765.tar.gz | |
ACPI: get_throttling_state() cannot be larger than state_count
Reported-by: Roel Kluin <roel.kluin@gmail.com>
Acked-by: Zhao Yakui <yakui.zhao@intel.com>
Signed-off-by: Len Brown <len.brown@intel.com>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/acpi/processor_throttling.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/drivers/acpi/processor_throttling.c b/drivers/acpi/processor_throttling.c index d278381..5f09901 100644 --- a/drivers/acpi/processor_throttling.c +++ b/drivers/acpi/processor_throttling.c @@ -783,11 +783,9 @@ static int acpi_get_throttling_state(struct acpi_processor *pr, (struct acpi_processor_tx_tss *)&(pr->throttling. states_tss[i]); if (tx->control == value) - break; + return i; } - if (i > pr->throttling.state_count) - i = -1; - return i; + return -1; } static int acpi_get_throttling_value(struct acpi_processor *pr, |
