mbox series

[v2,0/4] KEYS: x509: various cleanups

Message ID 20220119005436.119072-1-ebiggers@kernel.org (mailing list archive)
Headers show
Series KEYS: x509: various cleanups | expand

Message

Eric Biggers Jan. 19, 2022, 12:54 a.m. UTC 
This series cleans up a few things in the X.509 certificate parser.

Changed v1 => v2:
  - Renamed label in patch 3
  - Added Acked-by's

Eric Biggers (4):
  KEYS: x509: clearly distinguish between key and signature algorithms
  KEYS: x509: remove unused fields
  KEYS: x509: remove never-set ->unsupported_key flag
  KEYS: x509: remove dead code that set ->unsupported_sig

 crypto/asymmetric_keys/pkcs7_verify.c     |  7 ++---
 crypto/asymmetric_keys/x509.asn1          |  2 +-
 crypto/asymmetric_keys/x509_cert_parser.c | 34 ++++++++++++-----------
 crypto/asymmetric_keys/x509_parser.h      |  1 -
 crypto/asymmetric_keys/x509_public_key.c  | 18 ------------
 5 files changed, 21 insertions(+), 41 deletions(-)

Comments

Jarkko Sakkinen Jan. 26, 2022, 2:16 p.m. UTC | #1 
On Tue, Jan 18, 2022 at 04:54:32PM -0800, Eric Biggers wrote:
> This series cleans up a few things in the X.509 certificate parser.
> 
> Changed v1 => v2:
>   - Renamed label in patch 3
>   - Added Acked-by's
> 
> Eric Biggers (4):
>   KEYS: x509: clearly distinguish between key and signature algorithms
>   KEYS: x509: remove unused fields
>   KEYS: x509: remove never-set ->unsupported_key flag
>   KEYS: x509: remove dead code that set ->unsupported_sig
> 
>  crypto/asymmetric_keys/pkcs7_verify.c     |  7 ++---
>  crypto/asymmetric_keys/x509.asn1          |  2 +-
>  crypto/asymmetric_keys/x509_cert_parser.c | 34 ++++++++++++-----------
>  crypto/asymmetric_keys/x509_parser.h      |  1 -
>  crypto/asymmetric_keys/x509_public_key.c  | 18 ------------
>  5 files changed, 21 insertions(+), 41 deletions(-)
> 
> -- 
> 2.34.1
> 

I'll apply these (with ackd -> reviewed).


/Jarkko
Jarkko Sakkinen Jan. 26, 2022, 2:19 p.m. UTC | #2 
On Wed, Jan 26, 2022 at 04:17:01PM +0200, Jarkko Sakkinen wrote:
> On Tue, Jan 18, 2022 at 04:54:32PM -0800, Eric Biggers wrote:
> > This series cleans up a few things in the X.509 certificate parser.
> > 
> > Changed v1 => v2:
> >   - Renamed label in patch 3
> >   - Added Acked-by's
> > 
> > Eric Biggers (4):
> >   KEYS: x509: clearly distinguish between key and signature algorithms
> >   KEYS: x509: remove unused fields
> >   KEYS: x509: remove never-set ->unsupported_key flag
> >   KEYS: x509: remove dead code that set ->unsupported_sig
> > 
> >  crypto/asymmetric_keys/pkcs7_verify.c     |  7 ++---
> >  crypto/asymmetric_keys/x509.asn1          |  2 +-
> >  crypto/asymmetric_keys/x509_cert_parser.c | 34 ++++++++++++-----------
> >  crypto/asymmetric_keys/x509_parser.h      |  1 -
> >  crypto/asymmetric_keys/x509_public_key.c  | 18 ------------
> >  5 files changed, 21 insertions(+), 41 deletions(-)
> > 
> > -- 
> > 2.34.1
> > 
> 
> I'll apply these (with ackd -> reviewed).

Done

BR, Jarkko