On 07/19/2013 04:43 PM, Lukas Slebodnik wrote:
On (19/07/13 15:40), Ondrej Kos wrote:
> Hi,
>
> Attached find patch addressing issue
>
https://fedorahosted.org/sssd/ticket/2007
>
> Ondra
> --
> Ondrej Kos
> Associate Software Engineer
> Identity Management - SSSD
> Red Hat Czech
>From 1ccad686fc0c4857a7f0ca73fb39c3625673129e Mon Sep 17 00:00:00 2001
> From: Ondrej Kos <okos(a)redhat.com>
> Date: Fri, 19 Jul 2013 15:37:03 +0200
> Subject: [PATCH] TOOLS: Include PAC responder when rising debuglevel
>
>
https://fedorahosted.org/sssd/ticket/2007
> ---
> src/tools/sss_debuglevel.c | 6 ++++--
> 1 file changed, 4 insertions(+), 2 deletions(-)
>
> diff --git a/src/tools/sss_debuglevel.c b/src/tools/sss_debuglevel.c
> index
ce428bd9209f46fb2dd0d926bd2cab8decd6c2be..d5485a4b167cdb7e9ef87291e4b8c03c84833845 100644
> --- a/src/tools/sss_debuglevel.c
> +++ b/src/tools/sss_debuglevel.c
> @@ -238,9 +238,11 @@ errno_t get_confdb_sections(TALLOC_CTX *ctx, struct confdb_ctx
*confdb,
> const char *known_services[] = {
> CONFDB_MONITOR_CONF_ENTRY,
> CONFDB_NSS_CONF_ENTRY,
> - CONFDB_PAM_CONF_ENTRY
> + CONFDB_PAM_CONF_ENTRY,
> + CONFDB_PAC_CONF_ENTRY
> };
Could you also add sudo, autofs and ssh?
I was thinking about that initially, but
with bugzilla addressing only
PAC, i did only that one. Updated
> - static const int known_services_count = 3;
> + static const int known_services_count = sizeof(known_services)
> + / sizeof(const char *);
^^^^^^^^^^^^
I would prefer either either known_services[0] or *known_services
> TALLOC_CTX *tmp_ctx = talloc_new(NULL);
>
> if (tmp_ctx == NULL) {
> --
LS
_______________________________________________
sssd-devel mailing list
sssd-devel(a)lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/sssd-devel
New patch is attached.
Ondra
--
Ondrej Kos
Associate Software Engineer
Identity Management - SSSD
Red Hat Czech