[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PULL v1 00/18] Merge crypto 201/07/18
From: |
Daniel P. Berrange |
Subject: |
Re: [Qemu-devel] [PULL v1 00/18] Merge crypto 201/07/18 |
Date: |
Wed, 19 Jul 2017 09:42:41 +0100 |
User-agent: |
Mutt/1.8.3 (2017-05-23) |
On Wed, Jul 19, 2017 at 09:28:01AM +0100, Daniel P. Berrange wrote:
> On Wed, Jul 19, 2017 at 09:11:17AM +0100, Peter Maydell wrote:
> > On 18 July 2017 at 11:25, Daniel P. Berrange <address@hidden> wrote:
> > > The following changes since commit
> > > 6c6076662d98c068059983d411cb2a8987ba5670:
> > >
> > > Merge remote-tracking branch 'remotes/bonzini/tags/for-upstream' into
> > > staging (2017-07-14 12:16:09 +0100)
> > >
> > > are available in the git repository at:
> > >
> > > git://github.com/berrange/qemu tags/pull-qcrypto-2017-07-18-1
> > >
> > > for you to fetch changes up to 0e0fb56bcc8145d956bf527bfedc2be42bca101f:
> > >
> > > tests: crypto: add hmac speed benchmark support (2017-07-18 11:18:30
> > > +0100)
> > >
> > > ----------------------------------------------------------------
> > > Merge crypto 2017/07/18 v1
> > >
> > > ----------------------------------------------------------------
> >
> > Hi; I'm afraid this failed to build on clang (x86 Linux, FreeBSD,
> > OSX):
> >
> > CC crypto/cipher.o
> > /home/petmay01/linaro/qemu-for-merges/crypto/cipher.c:177:9: error:
> > variable 'drv' is used uninitialized whenever 'if' condition is false
> > [-Werror,-Wsometimes-uninitialized]
> > if (!ctx) {
> > ^~~~
> > /home/petmay01/linaro/qemu-for-merges/crypto/cipher.c:192:30: note:
> > uninitialized use occurs here
> > cipher->driver = (void *)drv;
> > ^~~
> > /home/petmay01/linaro/qemu-for-merges/crypto/cipher.c:177:5: note:
> > remove the 'if' if its condition is always true
> > if (!ctx) {
> > ^~~~~~~~~~
> > /home/petmay01/linaro/qemu-for-merges/crypto/cipher.c:168:29: note:
> > initialize the variable 'drv' to silence this warning
> > QCryptoCipherDriver *drv;
> > ^
> > = NULL
> > 1 error generated.
> >
> > and also
> >
> > CC crypto/hmac.o
> > /home/petmay01/linaro/qemu-for-merges/crypto/hmac.c:103:9: error:
> > variable 'drv' is used uninitialized whenever 'if' condition is false
> > [-Werror,-Wsometimes-uninitialized]
> > if (!ctx) {
> > ^~~~
> > /home/petmay01/linaro/qemu-for-merges/crypto/hmac.c:116:28: note:
> > uninitialized use occurs here
> > hmac->driver = (void *)drv;
> > ^~~
> > /home/petmay01/linaro/qemu-for-merges/crypto/hmac.c:103:5: note:
> > remove the 'if' if its condition is always true
> > if (!ctx) {
> > ^~~~~~~~~~
> > /home/petmay01/linaro/qemu-for-merges/crypto/hmac.c:94:27: note:
> > initialize the variable 'drv' to silence this warning
> > QCryptoHmacDriver *drv;
> > ^
> > = NULL
> >
> >
> > Looks like a false positive, I think :-(
>
> Ok, I'll investigate.
>
> BTW, any idea what's different between your clang builds and the
> clang builds that travis does - the latter passed on both OS-X
> and Linux.
Never mind, I just it must just be a newer version of clang - the travis
clang build uses 3.4.0 on Linux and 7.3.0 (?!?!??) on OS-X, where as latest
I see for Linux is 4.0.0
/me wonders how OS-X clang versions map to Lnux clang versions ??!
Regards,
Daniel
--
|: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org -o- https://fstop138.berrange.com :|
|: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|
- [Qemu-devel] [PULL v1 11/18] crypto: hmac: add hmac driver framework, (continued)
- [Qemu-devel] [PULL v1 11/18] crypto: hmac: add hmac driver framework, Daniel P. Berrange, 2017/07/18
- [Qemu-devel] [PULL v1 13/18] crypto: cipher: add afalg-backend cipher support, Daniel P. Berrange, 2017/07/18
- [Qemu-devel] [PULL v1 12/18] crypto: introduce some common functions for af_alg backend, Daniel P. Berrange, 2017/07/18
- [Qemu-devel] [PULL v1 15/18] crypto: hmac: add af_alg-backend hmac support, Daniel P. Berrange, 2017/07/18
- [Qemu-devel] [PULL v1 16/18] tests: crypto: add cipher speed benchmark support, Daniel P. Berrange, 2017/07/18
- [Qemu-devel] [PULL v1 14/18] crypto: hash: add afalg-backend hash support, Daniel P. Berrange, 2017/07/18
- [Qemu-devel] [PULL v1 17/18] tests: crypto: add hash speed benchmark support, Daniel P. Berrange, 2017/07/18
- [Qemu-devel] [PULL v1 18/18] tests: crypto: add hmac speed benchmark support, Daniel P. Berrange, 2017/07/18
- Re: [Qemu-devel] [PULL v1 00/18] Merge crypto 201/07/18, Peter Maydell, 2017/07/19