URL: https://github.com/SSSD/sssd/pull/94 Title: #94: Enable {socket,dbus}-activation for responders
fidencio commented: """ Patchset updated! I had to do some changes in a few commits in order to fix a regression introduced by the previous series. Let me try to sum up the changes: - SBUS: Add destructor data to sbus_connection: Instead of passing a function pointer to be registered as the destructor function, just pass the data that is required by the destructor function and set up the function whenever it's needed (in our case, in the monitor ... but it's part of another patch) - MONITOR: Unset conn destructor in the svc destructor: dropped as this patch (and the previous one) were causing an invalid access to the memory. So, I've decided to take a simpler approach (well, at least it looks simpler from my POV). - RESPONDER: Shutdown {dbus,socket}-activated responders in case they're idle: Now it takes care of setting the destructor for the sbus_connection and unsetting it in order to avoid a possible double-free. Hopefully this version is good enough to be pushed. """ See the full comment at https://github.com/SSSD/sssd/pull/94#issuecomment-273976309
_______________________________________________ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org