From mboxrd@z Thu Jan 1 00:00:00 1970 From: Darren Hart Date: Sat, 14 Mar 2015 19:03:28 +0000 Subject: Re: [patch 2/2] thinkpad_acpi: off by one in adaptive_keyboard_hotkey_notify_hotkey() Message-Id: <20150314190328.GF16816@fury.dvhart.com> List-Id: References: <20150311093607.GB3564@mwanda> In-Reply-To: <20150311093607.GB3564@mwanda> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Dan Carpenter Cc: Henrique de Moraes Holschuh , Bastien Nocera , ibm-acpi-devel@lists.sourceforge.net, platform-driver-x86@vger.kernel.org, kernel-janitors@vger.kernel.org On Wed, Mar 11, 2015 at 12:36:07PM +0300, Dan Carpenter wrote: > This should be >= instead of > because otherwise we read one element > past the end of the hotkey_keycode_map[] array. > > The hotkey_keycode_map[] array has TPACPI_HOTKEY_MAP_LEN elements. > > Fixes: 6a68d8557084 ('thinkpad_acpi: Add support for more adaptive kbd buttons') > Signed-off-by: Dan Carpenter Queued, thanks. -- Darren Hart Intel Open Source Technology Center