[libvirt] [PATCH 05/10] storage: avoid s[n]printf

Matthias Bolte matthias.bolte at googlemail.com
Thu Aug 19 21:10:04 UTC 2010


2010/8/19 Eric Blake <eblake at redhat.com>:
> * src/storage/storage_backend.c (virStorageBackendCreateQemuImg)
> (virStorageBackendCreateQcowCreate): Use virAsprintf instead.
> * src/storage/storage_backend_disk.c
> (virStorageBackendDiskCreateVol, virStorageBackendDiskPartFormat):
> Likewise.
> ---
>
> Things to look out for:
> virStorageBackendDiskPartFormat can now fail where it used to
> do nothing to the passed-in partFormat variable, if the switch
> statement hits the default.
>

Nice one, before it would just have executed parted with a random
string in the argumentlist in that case.

ACK.

Matthias




More information about the libvir-list mailing list