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

Jakub Hrozek jhrozek at redhat.com
Fri Jul 19 15:53:24 UTC 2013


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.



More information about the sssd-devel mailing list