Re: [PATCH v2] ACPI: Fixes device power states array overflow

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Jun 1, 2011 at 11:54 PM, Lin Ming <ming.m.lin@xxxxxxxxx> wrote:
> Commit 28c2103 added new state ACPI_STATE_D3_COLD, so the device power
> states array must be expanded by one also.
>
> v2: Use ACPI_D_STATE_COUNT instead of number 5 for the array size.
>
> Reported-by: Dan Carpenter <error27@xxxxxxxxx>
> Suggested-by: OldÅich JedliÄka <oldium.pro@xxxxxxxxx>
> Signed-off-by: Lin Ming <ming.m.lin@xxxxxxxxx>
> ---
> Âinclude/acpi/acpi_bus.h | Â Â2 +-
> Â1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/include/acpi/acpi_bus.h b/include/acpi/acpi_bus.h
> index 3a10ef5..6cd5b64 100644
> --- a/include/acpi/acpi_bus.h
> +++ b/include/acpi/acpi_bus.h
> @@ -210,7 +210,7 @@ struct acpi_device_power_state {
> Âstruct acpi_device_power {
> Â Â Â Âint state; Â Â Â Â Â Â Â/* Current state */
> Â Â Â Âstruct acpi_device_power_flags flags;
> - Â Â Â struct acpi_device_power_state states[4]; Â Â Â /* Power states (D0-D3) */
> + Â Â Â struct acpi_device_power_state states[ACPI_D_STATE_COUNT]; Â Â Â/* Power states (D0-D3Cold) */
> Â};
>
> Â/* Performance Management */
> --
> 1.7.2.3

Len,

Would you apply this one?

Thanks,
Lin Ming
--
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux IBM ACPI]     [Linux Power Management]     [Linux Kernel]     [Linux Laptop]     [Kernel Newbies]     [Share Photos]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]     [Linux Resources]

  Powered by Linux