Hi Daniel,
thanks for looking into this.
+ virConnectPtr conn = virConnectOpen(getuid() ?
+ "qemu:///session" :
+ "qemu:///system");
+
+ if (!conn) {
+ qemudLog(QEMUD_ERR, "%s",
+ _("Cannot autostart domains, failed to open
connection")); + return;
+ }
I thought about that check too, but decided against it, because
a) currently we always call it with conn=NULL, it only fails if you don't use
a directly bridged network
b) it currently won't do any harm calling with conn=NULL, creating the vm
fails later on because it can't find the network device.
But if some code relying on the conn-structure is added later on without
sanity-checks, this check will make sure no bigger harm is done.
Kind regards,
Gerd
--
Address (better: trap) for people I really don't want to get mail from:
james(a)cactusamerica.com