Received: by 2002:a25:86ce:0:0:0:0:0 with SMTP id y14csp1397439ybm; Wed, 22 May 2019 23:52:17 -0700 (PDT) X-Google-Smtp-Source: APXvYqwxEOymMi1VmGmjriWGbYkfSc0vxoQRHdHdDWoTMcdV+vAd98kODwwJBwhhBRjgHBnIlhoO X-Received: by 2002:a17:902:b58f:: with SMTP id a15mr52188416pls.201.1558594337704; Wed, 22 May 2019 23:52:17 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1558594337; cv=none; d=google.com; s=arc-20160816; b=ximTja1KlnQ030OmnjaCJ5RpiF/4ZyvgbRPvCWCW6QCr8pdLZlzWQS0ddOiHDnAoyI Vy/tWiZy5aQZpwPaq90BJ8BsZkqTi6RwNSgoHaCL3Al4FPGYamcrx1mOF8Lhh2WaPR1r u282v9Cgyn5zjfUgq2HKNgKRQREXkduSPXkNxLG35/GjfueIthyRD32p2d4cQ/WGtlwf yM3uYU+LXUSvFiJEJDyhrkzfqbXPeGXzRYU7DZxk7irH4242mo1Uk45mG0Kw52yAqox6 vKhoZUfY0vcJcrE2qyef7YzCbZliLhpTydSYD5m07LHJV2zViE5hRdT9wrGDc+ADQO+r 2erw== 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=X/rDYQqMTG4FALEqo87OF6Nk5YuJEztC3LL0gilqH5U=; b=FRwFo9JGw3Lzk6yK/lAdPSU90uaV+OaNRn9o98VRnxzSYdq/4dWHsaA7souxlMOxue fLTQUNpRik8OxzMAU1wQN5CoIFLkkVlBNJAQQ66iuMQn5vxg4lJL3WRNcwD+h6lMvoVV lRbHr8D9KnPUwcXopTDUoq6ee3i9jNAcvGDeJih9BUrdOQZsRYwGRMbCG3uDOWv4Mo2j 9DDFox8SBiIltO6QvOsnQkGRax1lH+mhqnHcPv8VbtvfTXcNiGv+3eMBlsUJhbQ4Sd2s kJI6oiB0uuSGw4lr9XRFMgY7NhXlK3LF0b8z3A8cyD3aHpCUuhISjD7DdK73gyuNByDw cglQ== 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 d126si26963546pgc.46.2019.05.22.23.52.01; Wed, 22 May 2019 23:52:17 -0700 (PDT) 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 S1725814AbfEWGvk (ORCPT + 99 others); Thu, 23 May 2019 02:51:40 -0400 Received: from helcar.hmeau.com ([216.24.177.18]:47792 "EHLO deadmen.hmeau.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726363AbfEWGvk (ORCPT ); Thu, 23 May 2019 02:51:40 -0400 Received: from gondobar.mordor.me.apana.org.au ([192.168.128.4] helo=gondobar) by deadmen.hmeau.com with esmtps (Exim 4.89 #2 (Debian)) id 1hThZh-0001oV-Lv; Thu, 23 May 2019 14:51:37 +0800 Received: from herbert by gondobar with local (Exim 4.89) (envelope-from ) id 1hThZe-0006BE-OW; Thu, 23 May 2019 14:51:34 +0800 Date: Thu, 23 May 2019 14:51:34 +0800 From: Herbert Xu To: Christophe Leroy Cc: "David S. Miller" , linux-crypto@vger.kernel.org, linux-kernel@vger.kernel.org, linuxppc-dev@lists.ozlabs.org Subject: Re: [PATCH] crypto: talitos - fix skcipher failure due to wrong output IV Message-ID: <20190523065134.hcvkalkpayywlonl@gondor.apana.org.au> References: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: NeoMutt/20170113 (1.7.2) Sender: linux-crypto-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-crypto@vger.kernel.org On Wed, May 15, 2019 at 12:29:03PM +0000, Christophe Leroy wrote: > Selftests report the following: > > [ 2.984845] alg: skcipher: cbc-aes-talitos encryption test failed (wrong output IV) on test vector 0, cfg="in-place" > [ 2.995377] 00000000: 3d af ba 42 9d 9e b4 30 b4 22 da 80 2c 9f ac 41 > [ 3.032673] alg: skcipher: cbc-des-talitos encryption test failed (wrong output IV) on test vector 0, cfg="in-place" > [ 3.043185] 00000000: fe dc ba 98 76 54 32 10 > [ 3.063238] alg: skcipher: cbc-3des-talitos encryption test failed (wrong output IV) on test vector 0, cfg="in-place" > [ 3.073818] 00000000: 7d 33 88 93 0f 93 b2 42 > > This above dumps show that the actual output IV is indeed the input IV. > This is due to the IV not being copied back into the request. > > This patch fixes that. > > Signed-off-by: Christophe Leroy > --- > drivers/crypto/talitos.c | 4 ++++ > 1 file changed, 4 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