virDomainBackupDiskDefParseXML: Fill default backup state after parsing it

Set the backup mode to VIR_TRISTATE_BOOL_YES after virXMLPropTristateBool
left it set to VIR_TRISTATE_BOOL_ABSENT. This will allow fixing
virXMLPropTristateBool to always initialize @result.

Signed-off-by: Peter Krempa <pkrempa@redhat.com>
Reviewed-by: Michal Privoznik <mprivozn@redhat.com>
This commit is contained in:
Peter Krempa 2021-05-06 16:01:32 +02:00
parent 23fdb5e3db
commit 46e6542a92

View File

@ -121,12 +121,13 @@ virDomainBackupDiskDefParseXML(xmlNodePtr node,
return -1;
}
def->backup = VIR_TRISTATE_BOOL_YES;
if (virXMLPropTristateBool(node, "backup", VIR_XML_PROP_NONE,
&def->backup) < 0)
return -1;
if (def->backup == VIR_TRISTATE_BOOL_ABSENT)
def->backup = VIR_TRISTATE_BOOL_YES;
/* don't parse anything else if backup is disabled */
if (def->backup == VIR_TRISTATE_BOOL_NO)
return 0;