[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 3/6] crypto: move QCryptoHashAlgorithm enum defi
From: |
Daniel P. Berrange |
Subject: |
Re: [Qemu-devel] [PATCH 3/6] crypto: move QCryptoHashAlgorithm enum definition into QAPI |
Date: |
Tue, 22 Dec 2015 15:50:22 +0000 |
User-agent: |
Mutt/1.5.24 (2015-08-30) |
On Mon, Dec 21, 2015 at 09:27:15AM -0700, Eric Blake wrote:
> On 12/21/2015 09:06 AM, Daniel P. Berrange wrote:
> > The QCryptoHashAlgorithm enum is defined in the crypto/hash.h
> > header. In the future some QAPI types will want to reference
> > the hash enums, so move the enum definition into QAPI too.
> >
> > Signed-off-by: Daniel P. Berrange <address@hidden>
> > ---
> > crypto/hash.c | 4 ++--
> > include/crypto/hash.h | 9 +--------
> > qapi/crypto.json | 15 +++++++++++++++
> > 3 files changed, 18 insertions(+), 10 deletions(-)
> >
> > diff --git a/crypto/hash.c b/crypto/hash.c
> > index 5a47b90..b5f81c4 100644
> > --- a/crypto/hash.c
> > +++ b/crypto/hash.c
> > @@ -24,13 +24,13 @@
> > #include <gnutls/gnutls.h>
> > #include <gnutls/crypto.h>
> >
> > -static int qcrypto_hash_alg_map[QCRYPTO_HASH_ALG_LAST] = {
> > +static int qcrypto_hash_alg_map[QCRYPTO_HASH_ALG_MAX] = {
>
> Will need rebasing, now that commit 7fb1cf1 has landed and renamed it
> QCRYPTO_HASH_ALG__MAX.
>
>
> > +++ b/qapi/crypto.json
> > @@ -33,3 +33,18 @@
> > { 'enum': 'QCryptoSecretFormat',
> > 'prefix': 'QCRYPTO_SECRET_FORMAT',
> > 'data': ['raw', 'base64']}
> > +
> > +
> > +##
> > +# QCryptoHashAlgorithm:
> > +#
> > +# The supported algorithms for computing content digests
> > +#
> > +# @md5: MD5. Should not be used in any new code, legacy compat only
> > +# @sha1: SHA-1. Should not be used in any new code, legacy compat only
> > +# @sha256: SHA-256. Current recommended strong hash.
> > +# Since: 2.6
> > +##
> > +{ 'enum': 'QCryptoHashAlgorithm',
> > + 'prefix': 'QCRYPTO_HASH_ALG',
> > + 'data': ['md5', 'sha1', 'sha256']}
>
> Otherwise looks fine. I know Markus is debating about getting rid of use
> of 'prefix', and this just makes that task harder, but the current
> munging would produce Q_CRYPTO_... even if you named the enum
> QCryptoHashAlg, so I'm okay with it.
If we do decide to kill "prefix" I'm happy to help cleanup the
fallout, since its all in code I've added...
> Reviewed-by: Eric Blake <address@hidden>
Regards,
Daniel
--
|: http://berrange.com -o- http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org -o- http://virt-manager.org :|
|: http://autobuild.org -o- http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org -o- http://live.gnome.org/gtk-vnc :|
- Re: [Qemu-devel] [PATCH 2/6] crypto: add ability to query hash digest len, (continued)