virCapabilitiesDomainDataLookup: Produce saner error message

During a review, I've noticed this error message that was eventually
produced when I was trying to define a domain:

error: invalid argument: could not find capabilities for arch=mips64el
domaintype=(null)

Look at the (null). Why is it there? Well, during XML parsing, we try
to look up the default emulator for given OS type and possibly virt
type too. And this is the problem, because if we don't want to look up
by virt type, a -1 is passed to note this fact. Later, the code
handles -1 just right. Except for error message. When it is
constructed (in a very fabulous way I must say), the value is compared
to zero, not -1. And since we don't have any translation from -1 to a
virt type string, we just print (null).

Signed-off-by: Michal Privoznik <mprivozn@redhat.com>
This commit is contained in:
Michal Privoznik 2015-06-11 16:15:46 +02:00
parent bcf1349bb1
commit f886701290

View File

@ -678,7 +678,7 @@ virCapabilitiesDomainDataLookupInternal(virCapsPtr caps,
virDomainOSTypeToString(ostype)); virDomainOSTypeToString(ostype));
if (arch) if (arch)
virBufferAsprintf(&buf, "arch=%s ", virArchToString(arch)); virBufferAsprintf(&buf, "arch=%s ", virArchToString(arch));
if (domaintype) if (domaintype != -1)
virBufferAsprintf(&buf, "domaintype=%s ", virBufferAsprintf(&buf, "domaintype=%s ",
virDomainVirtTypeToString(domaintype)); virDomainVirtTypeToString(domaintype));
if (emulator) if (emulator)