Re: [PATCH v2] crypto: AF_ALG - consolidation of duplicate code

2017-08-16 Thread Stephan Mueller
Am Mittwoch, 16. August 2017, 11:37:31 CEST schrieb Herbert Xu: Hi Herbert, > On Wed, Aug 16, 2017 at 10:39:43AM +0200, Stephan Mueller wrote: > > Are you feeding that patch to the Linus 4.13-rc tree via your crypto-2.6 > > tree? > Good point. Why don't you repost that patch and I'll push it > t

Re: [PATCH v2] crypto: AF_ALG - consolidation of duplicate code

2017-08-16 Thread Herbert Xu
On Wed, Aug 16, 2017 at 10:39:43AM +0200, Stephan Mueller wrote: > > Are you feeding that patch to the Linus 4.13-rc tree via your crypto-2.6 tree? Good point. Why don't you repost that patch and I'll push it to 4.13 along with stable. Thanks, -- Email: Herbert Xu Home Page: http://gondor.apan

Re: [PATCH v2] crypto: AF_ALG - consolidation of duplicate code

2017-08-16 Thread Stephan Mueller
Am Mittwoch, 16. August 2017, 10:29:18 CEST schrieb Herbert Xu: Hi Herbert, > On Thu, Aug 10, 2017 at 10:25:58AM +0200, Stephan Mueller wrote: > > I think there is such patch already, see [1]. > > > > Your comment to that patch triggered my rewrite of the memory managment > > code. > > > > [1]

Re: [PATCH v2] crypto: AF_ALG - consolidation of duplicate code

2017-08-16 Thread Herbert Xu
On Thu, Aug 10, 2017 at 10:25:58AM +0200, Stephan Mueller wrote: > > I think there is such patch already, see [1]. > > Your comment to that patch triggered my rewrite of the memory managment code. > > [1] https://www.spinics.net/lists/linux-crypto/msg21618.html Could you please post that patch t

Re: [PATCH v2] crypto: AF_ALG - consolidation of duplicate code

2017-08-10 Thread Stephan Mueller
Am Donnerstag, 10. August 2017, 10:21:53 CEST schrieb Herbert Xu: Hi Herbert, > On Thu, Aug 10, 2017 at 10:16:48AM +0200, Stephan Mueller wrote: > > As now the AIO code path is updated, the bug that I was reporting last > > September allowing to crash the kernel via AF_ALG is fixed. > > > > As t

Re: [PATCH v2] crypto: AF_ALG - consolidation of duplicate code

2017-08-10 Thread Herbert Xu
On Thu, Aug 10, 2017 at 10:16:48AM +0200, Stephan Mueller wrote: > > As now the AIO code path is updated, the bug that I was reporting last > September allowing to crash the kernel via AF_ALG is fixed. > > As the patch is very invasive, I am not sure that patch set should be sent to > stable. Ho

Re: [PATCH v2] crypto: AF_ALG - consolidation of duplicate code

2017-08-10 Thread Stephan Mueller
Am Mittwoch, 9. August 2017, 15:57:34 CEST schrieb Herbert Xu: Hi Herbert, > > Patch applied. Thanks. Thanks. As now the AIO code path is updated, the bug that I was reporting last September allowing to crash the kernel via AF_ALG is fixed. As the patch is very invasive, I am not sure that p

Re: [PATCH v2] crypto: AF_ALG - consolidation of duplicate code

2017-08-09 Thread Herbert Xu
On Wed, Aug 02, 2017 at 07:56:19AM +0200, Stephan Müller wrote: > Hi Herbert, > > as agreed, the individual patches from the first submission are now changed. > > After review of the changes I had to apply to algif_aead and algif_skcipher, > I saw that they are all in the category that you agreed

[PATCH v2] crypto: AF_ALG - consolidation of duplicate code

2017-08-01 Thread Stephan Müller
Hi Herbert, as agreed, the individual patches from the first submission are now changed. After review of the changes I had to apply to algif_aead and algif_skcipher, I saw that they are all in the category that you agreed that can be rolled into this patch. Though, I documented the changes so tha