subj says it all,
bug: https://fedorahosted.org/sssd/ticket/2924
I have compiled and run make check|intgcheck but "not" actively tested
this patch.
HTH,
Simo.
--
Simo Sorce * Red Hat, Inc * New York
Hi,
the attached patches are my proposal to fix
https://fedorahosted.org/sssd/ticket/2829
I haven't tested them past make check yet, because I'm not sure I like
them myself :) but at the same time I can't see a better way to keep
track of the servers and let callers set state of servers.
The most ugly thing so far IMO is the fo_internal_owner member. I would
prefer to instead have a fo_server_wrap structure that would be used for
the server_list, but I didn't want to do a large change before we agree
the refcount is a good idea at all.
The other ugly side-effect is that we need to be sure that nobody calls
talloc_free on the fo_server structure. Instead, only the parent context
can be freed (that's also what the first patch is about).
Hi,
I found this potential crash when trying to find another issue in the
failover code. To reproduce, just revert the changes to
src/providers/fail_over.c and run make check, you should see either a
crash or at least an error if you use valgrind.
Hello,
I decided to share this design document although it still a work in progress. Attached patches are just prove of concept and are very much work in progress. So far patches also defers from design in order in which secondary slices are generated.
Thanks for feedback on this early state of effort.
Bye.
https://fedorahosted.org/sssd/wiki/IdmapAutoAssignNewSlices
ehlo,
The first patch fixes regression introduced in 1.13.2.
https://fedorahosted.org/sssd/ticket/2493
I'm sorry I didn't catch it in review.
Fortunatelly there is a simple but annoying workaround.
Change user to "non-root" + remove old log files.
The second patch should fix ticket #2493 in different way.
LS
ehlo,
The main reason for these patch was to improve
recently added logging to hbac.
Side effect of these change is improvement for libldb
and libsemanage (6th patch)
4th patch is not API/ABI change because
such version has not beeen released yet.
If you do not like change in hbac callback
hbac_debug_fn_t then we should also remove
because it is too internal then we should
remove also the first two arguments.
"file", "line" also leaks internal data from libhbac.
Removing the first two arguments would be almost
consistent callbacks in libldb and libsemanage.
LS
ehlo,
reason is explained in commit message.
The intention for this patch was to a simplify spec file
(at least a little bit :-)
If we want to have some requirements for version of lib{ldb,tdb}
then it would be better to have minimal required version in
BuildRequires + configure detection of libraries.
But it might be outdated if we use some ver ldb features.
LS