[libvirt] [PATCH] Fix virDomainDeviceInfoIsSet() to check all struct fields

Laine Stump laine at laine.org
Tue May 15 16:03:25 UTC 2012


On 05/15/2012 11:19 AM, Daniel P. Berrange wrote:
> From: "Daniel P. Berrange" <berrange at redhat.com>
>
> The virDomainDeviceInfoIsSet API was only checking if an
> address or alias was set in the struct. Thus if only a
> rom bar setting / filename, boot index, or USB master
> value was set, they could be accidentally dropped when
> formatting XML
> ---
>  src/conf/domain_conf.c |   15 +++++++++++----
>  src/conf/domain_conf.h |    4 ++++
>  2 files changed, 15 insertions(+), 4 deletions(-)

Oops. ACK.

Good idea putting the cautionary note in the struct definition;
hopefully that will prevent the same omission in the future.




More information about the libvir-list mailing list