Reference state when using it as opaque

There should be one more reference because it is being kept in the list
of callbacks as an opaque.  We also unref it properly using
virObjectFreeCallback.

Signed-off-by: Martin Kletzander <mkletzan@redhat.com>

diff --git a/src/conf/object_event.c b/src/conf/object_event.c
index b859835b47a1e1b0151736b0a0bd48083d231a42..1ef8b58d10c3ea525ad39f779e265c4a58d4131f 100644 (file)
--- a/src/conf/object_event.c
+++ b/src/conf/object_event.c
@@ -870,12 +870,16 @@ virObjectEventStateRegisterID(virConnectPtr conn,
         (state->timer = virEventAddTimeout(-1,
                                            virObjectEventTimer,
                                            state,
-                                           NULL)) < 0) {
+                                           virObjectFreeCallback)) < 0) {
         virReportError(VIR_ERR_INTERNAL_ERROR, "%s",
                        _("could not initialize domain event timer"));
         goto cleanup;
     }
 
+    /* event loop has one reference, but we need one more for the
+     * timer's opaque argument */
+    virObjectRef(state);
+
     ret = virObjectEventCallbackListAddID(conn, state->callbacks,
                                           key, filter, filter_opaque,
                                           klass, eventID,