[libvirt] [libvirt-glib] Add API to redefine an existing domain
Marc-André Lureau
marcandre.lureau at gmail.com
Mon Nov 21 19:27:17 UTC 2011
Hi
On Mon, Nov 21, 2011 at 6:53 PM, Zeeshan Ali (Khattak)
<zeeshanak at gnome.org> wrote:
> + g_return_val_if_fail(error == NULL || *error == NULL, FALSE);
This is wrong, it should be error != NULL && *error == NULL.
> + if (virDomainDefineXML(conn, xml) == NULL) {
> + if (error != NULL)
> + *error = gvir_error_new_literal(GVIR_DOMAIN_ERROR,
> + 0,
> + "Failed to set "
> + "domain configuration");
> + return FALSE;
> + }
Can you please verify that the return value is safe to ignore?
I would really prefer we add a runtime check that verifiy the handle
is the same as the one currently associated with the domain.
--
Marc-André Lureau
More information about the libvir-list
mailing list