From patchwork Wed Mar 4 19:55:27 2009 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Andrew Morton X-Patchwork-Id: 9908 Received: from vger.kernel.org (vger.kernel.org [209.132.176.167]) by demeter.kernel.org (8.14.2/8.14.2) with ESMTP id n24JuAr5023685 for ; Wed, 4 Mar 2009 19:56:15 GMT Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756194AbZCDT4O (ORCPT ); Wed, 4 Mar 2009 14:56:14 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1756160AbZCDT4O (ORCPT ); Wed, 4 Mar 2009 14:56:14 -0500 Received: from smtp1.linux-foundation.org ([140.211.169.13]:39332 "EHLO smtp1.linux-foundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756194AbZCDT4K (ORCPT ); Wed, 4 Mar 2009 14:56:10 -0500 Received: from imap1.linux-foundation.org (imap1.linux-foundation.org [140.211.169.55]) by smtp1.linux-foundation.org (8.14.2/8.13.5/Debian-3ubuntu1.1) with ESMTP id n24JtR4Z010538 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=NO); Wed, 4 Mar 2009 11:55:28 -0800 Received: from localhost.localdomain (localhost [127.0.0.1]) by imap1.linux-foundation.org (8.13.5.20060308/8.13.5/Debian-3ubuntu1.1) with ESMTP id n24JtRIO027997; Wed, 4 Mar 2009 11:55:27 -0800 Message-Id: <200903041955.n24JtRIO027997@imap1.linux-foundation.org> Subject: [patch 4/8] acpi: get_throttling_state() cannot be larger than state_count To: lenb@kernel.org Cc: linux-acpi@vger.kernel.org, akpm@linux-foundation.org, roel.kluin@gmail.com, yakui.zhao@intel.com From: akpm@linux-foundation.org Date: Wed, 04 Mar 2009 11:55:27 -0800 X-Spam-Status: No, hits=-3.453 required=5 tests=AWL, BAYES_00, OSDL_HEADER_SUBJECT_BRACKETED X-Spam-Checker-Version: SpamAssassin 3.2.4-osdl_revision__1.47__ X-MIMEDefang-Filter: lf$Revision: 1.188 $ X-Scanned-By: MIMEDefang 2.63 on 140.211.169.13 Sender: linux-acpi-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-acpi@vger.kernel.org From: Roel Kluin With for(i = 0; i < n; i++) { ... } i reaches n. Signed-off-by: Roel Kluin Acked-by: Cc: Len Brown Signed-off-by: Andrew Morton --- drivers/acpi/processor_throttling.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff -puN drivers/acpi/processor_throttling.c~acpi-get_throttling_state-cannot-be-larger-state_count drivers/acpi/processor_throttling.c --- a/drivers/acpi/processor_throttling.c~acpi-get_throttling_state-cannot-be-larger-state_count +++ a/drivers/acpi/processor_throttling.c @@ -785,7 +785,7 @@ static int acpi_get_throttling_state(str if (tx->control == value) break; } - if (i > pr->throttling.state_count) + if (i >= pr->throttling.state_count) i = -1; return i; }