Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753883AbbKWNFv (ORCPT ); Mon, 23 Nov 2015 08:05:51 -0500 Received: from helcar.hengli.com.au ([209.40.204.226]:58604 "EHLO helcar.hengli.com.au" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753589AbbKWNFt (ORCPT ); Mon, 23 Nov 2015 08:05:49 -0500 Date: Mon, 23 Nov 2015 21:05:46 +0800 From: Herbert Xu To: LABBE Corentin Cc: davem@davemloft.net, linux-crypto@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 1/2] crypto: sahara: set nb_[in|out]_sg as signed int Message-ID: <20151123130546.GH7866@gondor.apana.org.au> References: <1447936699-26854-1-git-send-email-clabbe.montjoie@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1447936699-26854-1-git-send-email-clabbe.montjoie@gmail.com> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 944 Lines: 20 On Thu, Nov 19, 2015 at 01:38:17PM +0100, LABBE Corentin wrote: > The two unsigned int variables nb_in_sg and nb_out_sg can be assigned > signed value (-EINVAL) from sg_nents_for_len(). > Furthermore they are used only by dma_map_sg and dma_unmap_sg which wait > for an signed int, so they must be set as int. > > Fixes: 6c2b74d4774f ("crypto: sahara - check return value of sg_nents_for_len") > Reported-by: Dan Carpenter > Signed-off-by: LABBE Corentin Both patches applied. Thanks. -- Email: Herbert Xu Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/