[libvirt] [PATCH v4 2/2] remote: Move the call to remoteClientFreePrivateCallbacks from FreeFunc to CloseFunc
John Ferlan
jferlan at redhat.com
Mon Nov 20 21:28:18 UTC 2017
On 11/13/2017 08:07 AM, xinhua.Cao wrote:
> Still because of commit id 'fe8f1c8b' where we generate a REF for the
> Register and that's transparent to the consumer (e.g. how would they
> know they need to ensure that Deregister is called), thus the purpose of
> this patch is to find a way to Deregister if it's determined that the
> consumer hasn't by the time of the "last" REF we'd have.
> This solution to this problem is to alter the processing to have the
> remoteClientCloseFunc handle performing the Deregister calls instead of
> the remoteClientFreeFunc because there's no way FreeFunc would be called
> unless the Deregister was already called.
> ---
> daemon/remote.c | 10 ++++++----
> 1 file changed, 6 insertions(+), 4 deletions(-)
>
Reviewed-by: John Ferlan <jferlan at redhat.com>
John
NB: Prior to pushing I want to give enough time for anyone else to pipe
in over the implementation. I know Martin had expressed some interest -
so it's essentially time to allow him to register any objections... I
figure at least 24 hours should be good enough, but it could be
Wednesday too...
More information about the libvir-list
mailing list