> I think it's better to keep it just after the first debug
print as the purpose
> of this loop is to print detailed data about arguments passed to
> virConnectBaselineCPU(). However, we shouldn't really crash there. I think
>
> xmlCPUs[i] ? xmlCPUs[i] : "(null)"
Well, in Linux, printf("%s", NULL) is generally safe (it prints
"(null").
However, that is not portable, so there is a wrapper or something in the
libvirt code that does "safe" printing of NULL pointers. I can't quite
remember exactly what it's called, but take a look around.
Thanks for the hint, it's NULLSTR(s) macro in internal.h, I'll use that.
Jirka