From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932949AbcA3Rmz (ORCPT ); Sat, 30 Jan 2016 12:42:55 -0500 Received: from bombadil.infradead.org ([198.137.202.9]:50625 "EHLO bombadil.infradead.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932215AbcA3Rmx (ORCPT ); Sat, 30 Jan 2016 12:42:53 -0500 Date: Sat, 30 Jan 2016 09:42:39 -0800 From: Darren Hart To: Dan Carpenter Cc: Sreedhara DS , platform-driver-x86@vger.kernel.org, linux-kernel@vger.kernel.org, kernel-janitors@vger.kernel.org Subject: Re: [patch] intel_scu_ipcutil: underflow in scu_reg_access() Message-ID: <20160130174239.GC9269@malice.jf.intel.com> References: <20160126092425.GA15717@mwanda> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20160126092425.GA15717@mwanda> User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Jan 26, 2016 at 12:24:25PM +0300, Dan Carpenter wrote: > "count" is controlled by the user and it can be negative. Let's prevent > that by making it unsigned. You have to have CAP_SYS_RAWIO to call this > function so the bug is not as serious as it could be. > > Fixes: 5369c02d951a ('intel_scu_ipc: Utility driver for intel scu ipc') > Signed-off-by: Dan Carpenter Queued to fixes for 4.5-rc1 as well as stable. Thanks Dan, -- Darren Hart Intel Open Source Technology Center