[SSSD] [PATCH] TOOLS: Update all services with sss_debuglevel

Jakub Hrozek jhrozek at redhat.com
Wed Jul 31 09:33:07 UTC 2013


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 at 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 at 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



More information about the sssd-devel mailing list