On (20/07/16 11:10), Pavel Březina wrote: >CI: http://sssd-ci.duckdns.org/logs/job/50/01/summary.html > >The failure is about missing dependencies, unrelated to these patches. > >It depends on the sssctl failover patches due to changes in attaching dbus >message to a talloc context. Now it is possible to also free the message with >both dbus_message_unref() and talloc_free(). Since the sssctl patches are >already in late review process I didn't want to change them.
>From cc00161e956c998f2ae8e7b7aa42042f0a9d3b04 Mon Sep 17 00:00:00 2001 >From: =?UTF-8?q?Pavel=20B=C5=99ezina?= <pbrez...@redhat.com> >Date: Fri, 15 Jul 2016 14:50:41 +0200 >Subject: [PATCH 1/4] sbus: allow freeing msg through dbus api when using > talloc > >When a talloc-bound message was freed by removing all references >to it with dbus_message_unref we failed to free the talloc context >and thus leaking memory or unreferencing invalid message when >the parent context is freed. > >This patch allows to bound dbus message to talloc in the way that >allows us to free the message by both talloc and dbus api. >--- > src/sbus/sssd_dbus_utils.c | 51 ++++++++++++++++++++++++++++++++++++++++++++++ > 1 file changed, 51 insertions(+) > I think this patchset is not complete/ I cannot find the file src/sbus/sssd_dbus_utils.c in git master I was able to apply rest of patches but they cannot be compiled doe to undefined functions. BTW sorry for late review. LS _______________________________________________ sssd-devel mailing list sssd-devel@lists.fedorahosted.org https://lists.fedorahosted.org/admin/lists/sssd-devel@lists.fedorahosted.org