From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:57261) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bmd9E-0006v9-7r for qemu-devel@nongnu.org; Wed, 21 Sep 2016 04:44:57 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1bmd9D-0004Cd-3P for qemu-devel@nongnu.org; Wed, 21 Sep 2016 04:44:56 -0400 Date: Wed, 21 Sep 2016 09:44:43 +0100 From: "Daniel P. Berrange" Message-ID: <20160921084443.GK15535@redhat.com> Reply-To: "Daniel P. Berrange" References: <1474432046-325-1-git-send-email-famz@redhat.com> <1474432046-325-7-git-send-email-famz@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <1474432046-325-7-git-send-email-famz@redhat.com> Subject: Re: [Qemu-devel] [PATCH v9 06/13] crypto: Switch to QEMU UUID API List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Fam Zheng Cc: qemu-devel@nongnu.org, kwolf@redhat.com, qemu-block@nongnu.org, sw@weilnetz.de, jcody@redhat.com, mdroth@linux.vnet.ibm.com, armbru@redhat.com, pbonzini@redhat.com, mreitz@redhat.com, rth@twiddle.net On Wed, Sep 21, 2016 at 12:27:19PM +0800, Fam Zheng wrote: > The uuid generation doesn't return error, so update the function > signature and calling code accordingly. > > Signed-off-by: Fam Zheng > Reviewed-by: Eric Blake > Reviewed-by: Jeff Cody > --- > crypto/block-luks.c | 26 +++++++------------------- > 1 file changed, 7 insertions(+), 19 deletions(-) > > diff --git a/crypto/block-luks.c b/crypto/block-luks.c > index a848232..4530f82 100644 > --- a/crypto/block-luks.c > +++ b/crypto/block-luks.c > @@ -29,10 +29,7 @@ > #include "crypto/pbkdf.h" > #include "crypto/secret.h" > #include "crypto/random.h" > - > -#ifdef CONFIG_UUID > -#include > -#endif > +#include "qemu/uuid.h" > > #include "qemu/coroutine.h" > > @@ -877,18 +874,12 @@ qcrypto_block_luks_open(QCryptoBlock *block, > } > > > -static int > -qcrypto_block_luks_uuid_gen(uint8_t *uuidstr, Error **errp) > +static void > +qcrypto_block_luks_uuid_gen(uint8_t *uuidstr) > { > -#ifdef CONFIG_UUID > - uuid_t uuid; > - uuid_generate(uuid); > - uuid_unparse(uuid, (char *)uuidstr); > - return 0; > -#else > - error_setg(errp, "Unable to generate uuids on this platform"); > - return -1; > -#endif > + QemuUUID uuid; > + qemu_uuid_generate(&uuid); > + qemu_uuid_unparse(&uuid, (char *)uuidstr); > } > > static int > @@ -965,10 +956,7 @@ qcrypto_block_luks_create(QCryptoBlock *block, > * it out to disk > */ > luks->header.version = QCRYPTO_BLOCK_LUKS_VERSION; > - if (qcrypto_block_luks_uuid_gen(luks->header.uuid, > - errp) < 0) { > - goto error; > - } > + qcrypto_block_luks_uuid_gen(luks->header.uuid); > > cipher_alg = qcrypto_block_luks_cipher_alg_lookup(luks_opts.cipher_alg, > errp); Reviewed-by: Daniel P. Berrange 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 :|