On (12/08/16 10:35), Pavel Březina wrote:
>On 08/11/2016 04:03 PM, Lukas Slebodnik wrote:
>> 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?= <pbrezina(a)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
>
>Hi, thank you. It depends on "sssctl: print active server and server list".
>It is written in the first sentence.
The 1st sentence says "It depends on the sssctl failover patches"
Which I could not match with the thread :-)
Anyway, patches dod not broke anyting according to tests :-)
LGTM, because it's blocked by other thread.