From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932471AbdK2LFS (ORCPT ); Wed, 29 Nov 2017 06:05:18 -0500 Received: from mail.eperm.de ([89.247.134.16]:42670 "EHLO mail.eperm.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932287AbdK2LFQ (ORCPT ); Wed, 29 Nov 2017 06:05:16 -0500 From: Stephan =?ISO-8859-1?Q?M=FCller?= To: Herbert Xu Cc: Eric Biggers , syzbot , davem@davemloft.net, linux-crypto@vger.kernel.org, linux-kernel@vger.kernel.org, syzkaller-bugs@googlegroups.com Subject: Re: [PATCH v2] crypto: AF_ALG - race-free access of encryption flag Date: Wed, 29 Nov 2017 12:05:13 +0100 Message-ID: <2034920.efhAd224kL@positron.chronox.de> In-Reply-To: <20171129071049.GA23023@gondor.apana.org.au> References: <001a113f2cd2d62b59055efb7618@google.com> <2865588.MNjhcg78tp@tauon.chronox.de> <20171129071049.GA23023@gondor.apana.org.au> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Am Mittwoch, 29. November 2017, 08:10:49 CET schrieb Herbert Xu: Hi Herbert, > > It sort of worked for skcipher because it didn't care if ctx->enc > or even ctx->iv changed midstream. But even there I don't think > we need to wait a second time. In fact waiting a second time could > result in a dead-lock if no sendmsg call came around. Shouldn't we then create a patch for the pre-4.14 algif_skcipher code that moves the wait out of the while loop to the beginning of the function in recvmsg? Ciao Stephan