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

["Oldřich Jedlička" <oldium.pro@seznam.cz>]

Hi Lin,

On Monday 30 May 2011 at 15:42:41, Lin Ming wrote:
> Commit 28c2103 added new state ACPI_STATE_D3_COLD, so the device power
> states array must be expanded by one also.
> 
> Reported-by: Dan Carpenter <error27@gmail.com>
> Signed-off-by: Lin Ming <ming.m.lin@intel.com>
> ---
>  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..ff246e8 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[5];	/* Power states (D0-D3Cold) 
*/

Wouldn't it be better to use some constant, so that the same problem would not 
appear again in future - something like ACPI_D_STATE_COUNT or (maybe better) 
ACPI_D_STATES_MAX+1 instead of magic number 5?

Cheers,
Oldřich.


>  };
> 
>  /* Performance Management */
--
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html