On Fri, Jul 19, 2013 at 05:53:24PM +0200, Jakub Hrozek wrote:
On Fri, Jul 19, 2013 at 05:02:02PM +0200, Lukas Slebodnik wrote:
> On (19/07/13 16:49), Ondrej Kos wrote:
> >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
> >
>
> Thank you.
>
> ACK
>
> LS
Pushed to master.
Also pushed to sssd-1-10