[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