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(a)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...