Re: [PATCH net] esp: select CRYPTO_SEQIV

2020-06-05 Thread Greg Kroah-Hartman
On Thu, Jun 04, 2020 at 12:23:22PM -0700, Eric Biggers wrote: > From: Eric Biggers > > Since CRYPTO_CTR no longer selects CRYPTO_SEQIV, it should be selected > by INET_ESP and INET6_ESP -- similar to CRYPTO_ECHAINIV. > > Fixes: f23efcbcc523 ("crypto: ctr - no longer needs CRYPTO_SEQIV") > Cc: Co

Re: [PATCH net] esp: select CRYPTO_SEQIV

2020-06-04 Thread Herbert Xu
On Thu, Jun 04, 2020 at 10:09:10PM -0700, Eric Biggers wrote: > > There's also a case where "seqiv" is used without counter mode: > > net/xfrm/xfrm_algo.c: > > { > .name = "rfc7539esp(chacha20,poly1305)", So select CRYPTO_SEQIV if CRYPTO_CTR || CRYPTO_CHACHA20POLY1305 and if th

Re: [PATCH net] esp: select CRYPTO_SEQIV

2020-06-04 Thread Eric Biggers
On Fri, Jun 05, 2020 at 10:29:56AM +1000, Herbert Xu wrote: > On Fri, Jun 05, 2020 at 10:28:58AM +1000, Herbert Xu wrote: > > > > Hmm, the selection list doesn't include CTR so just adding SEQIV > > per se makes no sense. I'm not certain that we really want to > > include every algorithm under the

Re: [PATCH net] esp: select CRYPTO_SEQIV

2020-06-04 Thread Herbert Xu
On Fri, Jun 05, 2020 at 10:28:58AM +1000, Herbert Xu wrote: > > Hmm, the selection list doesn't include CTR so just adding SEQIV > per se makes no sense. I'm not certain that we really want to > include every algorithm under the sun. Steffen, what do you think? Or how about select CRYPT

Re: [PATCH net] esp: select CRYPTO_SEQIV

2020-06-04 Thread Herbert Xu
On Thu, Jun 04, 2020 at 12:23:22PM -0700, Eric Biggers wrote: > From: Eric Biggers > > Since CRYPTO_CTR no longer selects CRYPTO_SEQIV, it should be selected > by INET_ESP and INET6_ESP -- similar to CRYPTO_ECHAINIV. > > Fixes: f23efcbcc523 ("crypto: ctr - no longer needs CRYPTO_SEQIV") > Cc: Co

[PATCH net] esp: select CRYPTO_SEQIV

2020-06-04 Thread Eric Biggers
From: Eric Biggers Since CRYPTO_CTR no longer selects CRYPTO_SEQIV, it should be selected by INET_ESP and INET6_ESP -- similar to CRYPTO_ECHAINIV. Fixes: f23efcbcc523 ("crypto: ctr - no longer needs CRYPTO_SEQIV") Cc: Corentin Labbe Cc: Greg Kroah-Hartman Cc: Herbert Xu Cc: Steffen Klassert