Re: [PATCH] wireless: airo: switch to skcipher interface

2019-06-17 Thread Ard Biesheuvel
On Sun, 16 Jun 2019 at 21:43, Eric Biggers wrote: > > On Sun, Jun 16, 2019 at 09:03:58PM +0200, Ard Biesheuvel wrote: > > > > > > Otherwise this patch looks correct to me. > > > > > > The actual crypto in this driver, on the other hand, looks very outdated > > > and > > > broken. Apparently it's

Re: [PATCH] wireless: airo: switch to skcipher interface

2019-06-16 Thread Johannes Berg
On Sun, 2019-06-16 at 00:12 -0700, Eric Biggers wrote: > > The actual crypto in this driver, on the other hand, looks very outdated and > broken. Apparently it's implementing some Cisco proprietary extension to WEP > that uses a universal hashing based MAC, where the hash key is generated from >

Re: [PATCH] wireless: airo: switch to skcipher interface

2019-06-16 Thread Eric Biggers
On Sun, Jun 16, 2019 at 09:03:58PM +0200, Ard Biesheuvel wrote: > > > > Otherwise this patch looks correct to me. > > > > The actual crypto in this driver, on the other hand, looks very outdated and > > broken. Apparently it's implementing some Cisco proprietary extension to > > WEP > > that uses

Re: [PATCH] wireless: airo: switch to skcipher interface

2019-06-16 Thread Ard Biesheuvel
On Sun, 16 Jun 2019 at 09:12, Eric Biggers wrote: > > On Fri, Jun 14, 2019 at 11:36:03AM +0200, Ard Biesheuvel wrote: > > The AIRO driver applies a ctr(aes) on a buffer of considerable size > > (2400 bytes), and instead of invoking the crypto API to handle this > > in its entirety, it open codes t

Re: [PATCH] wireless: airo: switch to skcipher interface

2019-06-16 Thread Eric Biggers
On Fri, Jun 14, 2019 at 11:36:03AM +0200, Ard Biesheuvel wrote: > The AIRO driver applies a ctr(aes) on a buffer of considerable size > (2400 bytes), and instead of invoking the crypto API to handle this > in its entirety, it open codes the counter manipulation and invokes > the AES block cipher di

Re: [PATCH] wireless: airo: switch to skcipher interface

2019-06-14 Thread Ard Biesheuvel
On Fri, 14 Jun 2019 at 11:42, Ard Biesheuvel wrote: > > The AIRO driver applies a ctr(aes) on a buffer of considerable size > (2400 bytes), and instead of invoking the crypto API to handle this > in its entirety, it open codes the counter manipulation and invokes > the AES block cipher directly. >

[PATCH] wireless: airo: switch to skcipher interface

2019-06-14 Thread Ard Biesheuvel
The AIRO driver applies a ctr(aes) on a buffer of considerable size (2400 bytes), and instead of invoking the crypto API to handle this in its entirety, it open codes the counter manipulation and invokes the AES block cipher directly. Let's fix this, by switching to the sync skcipher API instead.

[PATCH] wireless: airo: switch to skcipher interface

2019-06-14 Thread Ard Biesheuvel
The AIRO driver applies a ctr(aes) on a buffer of considerable size (2400 bytes), and instead of invoking the crypto API to handle this in its entirety, it open codes the counter manipulation and invokes the AES block cipher directly. Let's fix this, by switching to the sync skcipher API instead.