On 03/20/2014 11:28 AM, Cole Robinson wrote: > Right now, on failure, libvirt.py doesn't raise an exception and just > returns -1 to the user. > --- > libvirt-override.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libvirt-override.c b/libvirt-override.c > index 7f746ed..9fed299 100644 > --- a/libvirt-override.c > +++ b/libvirt-override.c > @@ -2326,7 +2326,7 @@ libvirt_virConnectGetCPUModelNames(PyObject *self ATTRIBUTE_UNUSED, > LIBVIRT_END_ALLOW_THREADS; > > if (c_retval == -1) > - return VIR_PY_INT_FAIL; > + return VIR_PY_NONE; > Incomplete. The error: label needs the same treatment. ACK with that fix. -- Eric Blake eblake redhat com +1-919-301-3266 Libvirt virtualization library http://libvirt.org
Attachment:
signature.asc
Description: OpenPGP digital signature
-- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list