Marek Marczykowski-Górecki wrote: > Otherwise domainCreateWithFlags via remote returns -1 as domid. > > Signed-off-by: Marek Marczykowski-Górecki <marma...@invisiblethingslab.com> > --- > src/libxl/libxl_driver.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/src/libxl/libxl_driver.c b/src/libxl/libxl_driver.c > index c95b387..ce3a99b 100644 > --- a/src/libxl/libxl_driver.c > +++ b/src/libxl/libxl_driver.c > @@ -2375,6 +2375,9 @@ libxlDomainCreateWithFlags(virDomainPtr dom, > } > > ret = libxlDomainStart(driver, vm, (flags & VIR_DOMAIN_START_PAUSED) != > 0, -1); > + if (ret < 0) > + goto cleanup; > + dom->id = vm->def->id; >
It looks at though this is the only callsite of libxlDomainStart that needs this change. Have you noticed this with other APIs that call libxlDomainStart? ACK to the fix here though; pushed. Thanks! Regards, Jim > > cleanup: > if (vm) > -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list
