[libvirt] [PATCH] util: storage: Fix build after 25924dec0f9329d429aadae14e273602307e2214

Eric Blake eblake at redhat.com
Wed Jul 9 13:01:55 UTC 2014

On 07/09/2014 02:05 AM, Peter Krempa wrote:
> The commit referenced above changed function arguments of
> virStorageFileGetMetadataFromBuf() but didn't tweak the
> ATTRIBUTE_NONNULL tied to them. This was caught by coverity as it
> actually obeys them. We disabled them for GCC and thus it didn't show
> up.

Yeah, because gcc 4.8 still misbehaves on the attribute (silently
"optimizing" code without flagging invalid uses).  There's been talk on
the gcc list about fixing it, although probably no sooner than 4.10

> Additionally in commit 3ea661deeabadc3c114dfb6f662b9fd17d714a01 I passed
> NULL to the backingFormat argument which was also marked as nonnull. Use
> a dummy int's address when the argument isn't supplied so that the code
> doesn't need to change much.
> ---
>  src/util/virstoragefile.c | 4 ++++
>  src/util/virstoragefile.h | 3 +--
>  2 files changed, 5 insertions(+), 2 deletions(-)


Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 604 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20140709/493501e0/attachment-0001.sig>

More information about the libvir-list mailing list