[PATCH][ding_libs] Add version symbol files
by Lukas Slebodnik
ehlo,
attached patch addresses ticket #2193
I have just a question about refarray/libref_array.sym
There is extern function ref_array_debug, which is not defined
in public header file (not exposed in public API), but it is used in ref_array
unit tests. It needs to be in global section because linker with fail to find
symbol. Should I add any comment to the file libref_array.sym or does anyone
better solution?
LS
10 years
KRB: check ccache directory for user's GID
by Pavel Reichl
Hello,
I noticed that debug message does not reflect condition.
I suppose it was intended to to check user's gid too.
Please have mercy if I'm wrong :-).
Regards,
Pavel Reichl
10 years
Use a shorter retry timeout for failed SRV
by Pavel Reichl
Hello,
please see attached patch.
While testing this patch I have noticed a little asymmetry:
While SSSD being offline I added SRV record so resolving could success:
Call of 'getent passwd user@ad-domain ' results in 'Cannot proceed,
provider is offline'.
but calling 'getent passwd user' will try to resolve services and ends
up online (then previous call will obviously succeed too).
Pavel Reichl
10 years
[PATCH] krb5_child: Fix use after free
by Lukas Slebodnik
ehlo,
debug_prg_name is used in debug_fn and it was allocated under
talloc context "kr". The variable "kr" was removed before the last debug
messages in function main. It is very little change that it will be overridden.
It is possible to see this issue with exported environment variable
TALLOC_FREE_FILL=255
patch is attached.
LS
10 years
[PATCH] Re-add the InfoPipe server plumbing
by Jakub Hrozek
Hi,
I've been working on re-adding a public DBus API to SSSD lately, based on
Stephen's InfoPipe code. While I'm not finished yet, the interface is also
important to Pavel's OpenLMI thesis, so I'll be sending the patches for
review as the individual subtasks are finished and rebased on top of Stef's
recent patches.
Attached are two patches that I think are ready to be reviewed and merged
with some minor exceptions or questions. The review would be a good
opportunity to solve those.
[PATCH 1/2] IFP: Re-add the InfoPipe server
This commit only adds the responder and the needed plumbing. No DBus
related code is in yet.
With this patch, I was wondering whether to build the code by default, or
only when --enable-experimental-features is set until the whole feature is
finished?
Also, with the current code, all responders spawn a client socket now, but
this responder listens on the system bus instead. I created
https://fedorahosted.org/sssd/ticket/2290 for this purpose -- Pavel, feel
free to take it.
[PATCH 2/2] IFP: Connect to the system bus
Adds the possibility for the InfoPipe responder to connect to the system
bus. At the moment, only a dummy method "Ping" is provided. The method only
accepts a single string parameter that has to be 'ping'.
To test, you can call the Ping method like this:
dbus-send --print-reply --system --dest=org.freedesktop.sssd.infopipe
/org/freedesktop/sssd/infopipe org.freedesktop.sssd.infopipe.Ping
'string:Ping'
Or test that the code can catch wrong options:
dbus-send --print-reply --system --dest=org.freedesktop.sssd.infopipe
/org/freedesktop/sssd/infopipe org.freedesktop.sssd.infopipe.Ping
'string:sdf'
I'll be sending next round of patches later this week. Hopefully this would
unblock Pavel's work.
10 years