Fix crash in libvirtd when events are registered & ACLs active
authorDaniel P. Berrange <berrange@redhat.com>
Fri, 27 Sep 2013 14:46:07 +0000 (15:46 +0100)
committerDaniel P. Berrange <berrange@redhat.com>
Fri, 27 Sep 2013 15:49:51 +0000 (16:49 +0100)
commit03288d0de6894e18c9be187e2ace0cc50f15ceaa
tree61d4c9f6df41315886000aab5be3aa83b64fe406
parent476d0e38af11f3ff50d85e3f7aecad4cd8208c76
Fix crash in libvirtd when events are registered & ACLs active

When a client disconnects from libvirtd, all event callbacks
must be removed. This involves running the public API

  virConnectDomainEventDeregisterAny

This code does not run in normal API dispatch context, so no
identity was set. The result was that the access control drivers
denied the attempt to deregister callbacks. The callbacks thus
continued to trigger after the client was free'd causing fairly
predictable use of free memory & a crash.

This can be triggered by any client with readonly access when
the ACL drivers are active.

Signed-off-by: Daniel P. Berrange <berrange@redhat.com>
(cherry picked from commit 8294aa0c1750dcb49d6345cd9bd97bf421580d8b)

Conflicts:
daemon/remote.c: int/size_t changes
daemon/remote.c