xen_common: Change xenConfigCopyStringInternal to use virConfGetValueString

Signed-off-by: Fabiano Fidêncio <fidencio@redhat.com>
Reviewed-by: John Ferlan <jferlan@redhat.com>
This commit is contained in:
Fabiano Fidêncio 2018-09-20 15:28:47 +02:00 committed by John Ferlan
parent efc29ab2e5
commit db343ca830

View File

@ -145,10 +145,13 @@ xenConfigCopyStringInternal(virConfPtr conf,
char **value,
int allowMissing)
{
virConfValuePtr val;
int rc;
*value = NULL;
if (!(val = virConfGetValue(conf, name))) {
if ((rc = virConfGetValueString(conf, name, value)) < 0)
return -1;
if (rc == 0) {
if (allowMissing)
return 0;
virReportError(VIR_ERR_INTERNAL_ERROR,
@ -156,20 +159,7 @@ xenConfigCopyStringInternal(virConfPtr conf,
return -1;
}
if (val->type != VIR_CONF_STRING) {
virReportError(VIR_ERR_INTERNAL_ERROR,
_("config value %s was not a string"), name);
return -1;
}
if (!val->str) {
if (allowMissing)
return 0;
virReportError(VIR_ERR_INTERNAL_ERROR,
_("config value %s was missing"), name);
return -1;
}
return VIR_STRDUP(*value, val->str);
return 1;
}