External/libvirt
1
0
Fork 0
mirror of https://gitlab.com/libvirt/libvirt.git synced 2024-12-25 23:25:24 +00:00
Code Issues Packages Projects Releases Wiki Activity

Remove a completely bogus reference increment in the Xen driver.

Browse Source 
xenUnifiedDomainEventRegister() calls out to
virDomainEventCallbackListAdd(), which increments the reference
count on the connection.  That is fine, but then
xenUnifiedDomainEventRegister() increments the usage count again,
leading to a usage count leak.  Remove the increment in the xen
register, and the UnrefConnect in the xen unregister.

Signed-off-by: Chris Lalancette <clalance@redhat.com>
This commit is contained in:
Chris Lalancette 2009-10-23 08:57:03 +02:00
parent 43a9249c95
commit 6940109b9a
1 changed files with 0 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/xen/xen_driver.c
View File

@ -1502,9 +1502,6 @@ xenUnifiedDomainEventRegister (virConnectPtr conn,
ret = virDomainEventCallbackListAdd(conn, priv->domainEventCallbacks, ret = virDomainEventCallbackListAdd(conn, priv->domainEventCallbacks,
callback, opaque, freefunc); callback, opaque, freefunc);
if (ret == 0)
conn->refs++;
xenUnifiedUnlock(priv); xenUnifiedUnlock(priv);
return (ret); return (ret);
} }
@ -1530,9 +1527,6 @@ xenUnifiedDomainEventDeregister (virConnectPtr conn,
ret = virDomainEventCallbackListRemove(conn, priv->domainEventCallbacks, ret = virDomainEventCallbackListRemove(conn, priv->domainEventCallbacks,
callback); callback);
if (ret == 0)
virUnrefConnect(conn);
xenUnifiedUnlock(priv); xenUnifiedUnlock(priv);
return ret; return ret;
} }