From mboxrd@z Thu Jan 1 00:00:00 1970 From: Herbert Xu Subject: Re: [PATCH] crypto: mediatek: don't return garbage err on successful return Date: Fri, 13 Jan 2017 00:39:53 +0800 Message-ID: <20170112163953.GD20085@gondor.apana.org.au> References: <20170103132122.26900-1-colin.king@canonical.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: "David S . Miller" , Matthias Brugger , Ryder Lee , linux-crypto@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-mediatek@lists.infradead.org, linux-kernel@vger.kernel.org To: Colin King Return-path: Content-Disposition: inline In-Reply-To: <20170103132122.26900-1-colin.king@canonical.com> Sender: linux-kernel-owner@vger.kernel.org List-Id: linux-crypto.vger.kernel.org On Tue, Jan 03, 2017 at 01:21:22PM +0000, Colin King wrote: > From: Colin Ian King > > In the case where keylen <= bs mtk_sha_setkey returns an uninitialized > return value in err. Fix this by returning 0 instead of err. > > Issue detected by static analysis with cppcheck. > > Signed-off-by: Colin Ian King Patch applied. Thanks. -- Email: Herbert Xu Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt