Received: by 2002:a25:c205:0:0:0:0:0 with SMTP id s5csp6308476ybf; Thu, 5 Mar 2020 17:49:06 -0800 (PST) X-Google-Smtp-Source: ADFU+vtI4Fs9viX1L9zbEvoqkyOWy40jVBsAOwdPAXorDDRX0L1+Ula82+mMCadXGniFX7PF2OA4 X-Received: by 2002:a9d:70c8:: with SMTP id w8mr684475otj.144.1583459345963; Thu, 05 Mar 2020 17:49:05 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1583459345; cv=none; d=google.com; s=arc-20160816; b=WCknt9lBG9+do5Ydu1u9pD0lKESq9xuhMCjBCJdNP+MKkNQttX2zxILBOpzlaylics A28u8jJyJwoLNLtJgIiKrE2CO6SkkS1ISpXuL3pRLf7dxbD7K8XiaZ5N546tP1jIHzqe Hk3S0iJRrJzJCOG+0XjJhU0ZWxwGi/6bp8+NfoQxqAJZtAJQW5jOr4CeDisik3To6CTN /tUj4tVDzkZkxSwrFfUKO+KmjclpxCZqe420B74tnTb+VvtAA1aKJc+vHiw/8pwodh1T IPAub7QdB1iU958LJEyq+pf5ax4XwVz53ht8vrqCDk28AcYZkb6InK52s+wxNBIGn4uf 522A== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:user-agent:in-reply-to :content-disposition:mime-version:references:message-id:subject:cc :to:from:date; bh=tcI9VyijXFqyt8h4t489IIjCjxqbRzfIq2Ci473bHzs=; b=mW2Z4a9fxrZ6lJDqS+w5EixjYzLfmdk7BsEB1MfWRIXwGxZEVxuHg8sj7LMA2tUSo2 Itw9HFmmkUc0u6MCmKu9o6Eo67n1NNxS7JJ7dA+J+9pUq+4ykV7ync7oS1UuyDsfqUiv hmDazkm0i0bVwb6XlbCAvY0GKV8DvAd1t1D3hhIuFuKHy3UoRxifAgwbOdbwAVcSJmRS m3XpHog2urz+HnIrhEvuPzahD7Dub8KpYM4uEUuezPzxWbbpk219Lj+zyB1DiJq+HkPz XOVBhmoedB6nSJQjBPRcjFOk/ym3auedALyWUIPD4rnT4hfg7IW0cTceZ/2kB+wg6kUB kXyA== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-crypto-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-crypto-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id x20si437992oic.75.2020.03.05.17.48.54; Thu, 05 Mar 2020 17:49:05 -0800 (PST) Received-SPF: pass (google.com: best guess record for domain of linux-crypto-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-crypto-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-crypto-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726191AbgCFBsy (ORCPT + 99 others); Thu, 5 Mar 2020 20:48:54 -0500 Received: from helcar.hmeau.com ([216.24.177.18]:45994 "EHLO fornost.hmeau.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726162AbgCFBsy (ORCPT ); Thu, 5 Mar 2020 20:48:54 -0500 Received: from gwarestrin.me.apana.org.au ([192.168.0.7] helo=gwarestrin.arnor.me.apana.org.au) by fornost.hmeau.com with smtp (Exim 4.89 #2 (Debian)) id 1jA26D-0005ix-9t; Fri, 06 Mar 2020 12:48:26 +1100 Received: by gwarestrin.arnor.me.apana.org.au (sSMTP sendmail emulation); Fri, 06 Mar 2020 12:48:25 +1100 Date: Fri, 6 Mar 2020 12:48:25 +1100 From: Herbert Xu To: Corentin Labbe Cc: ard.biesheuvel@linaro.org, catalin.marinas@arm.com, davem@davemloft.net, will@kernel.org, ebiggers@kernel.org, linux-arm-kernel@lists.infradead.org, linux-crypto@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v2] crypto: arm64: CE: implement export/import Message-ID: <20200306014825.GB30653@gondor.apana.org.au> References: <1582555661-25737-1-git-send-email-clabbe@baylibre.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1582555661-25737-1-git-send-email-clabbe@baylibre.com> User-Agent: Mutt/1.10.1 (2018-07-13) Sender: linux-crypto-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-crypto@vger.kernel.org On Mon, Feb 24, 2020 at 02:47:41PM +0000, Corentin Labbe wrote: > When an ahash algorithm fallback to another ahash and that fallback is > shaXXX-CE, doing export/import lead to error like this: > alg: ahash: sha1-sun8i-ce export() overran state buffer on test vector 0, cfg=\"import/export\" > > This is due to the descsize of shaxxx-ce being larger than struct shaxxx_state > off by an u32. > For fixing this, let's implement export/import which rip the finalize > variant instead of using generic export/import. > > Fixes: 6ba6c74dfc6b ("arm64/crypto: SHA-224/SHA-256 using ARMv8 Crypto Extensions") > Fixes: 2c98833a42cd ("arm64/crypto: SHA-1 using ARMv8 Crypto Extensions") > > Signed-off-by: Corentin Labbe > --- > Changes since v1: > - memcpy directly &sctx->sst instead of sctx. As suggested by Eric Biggers > > arch/arm64/crypto/sha1-ce-glue.c | 20 ++++++++++++++++++++ > arch/arm64/crypto/sha2-ce-glue.c | 23 +++++++++++++++++++++++ > 2 files changed, 43 insertions(+) Patch applied. Thanks. -- Email: Herbert Xu Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt