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

Lukas Slebodnik lslebodn at redhat.com
Fri Jul 19 15:02:02 UTC 2013


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



More information about the sssd-devel mailing list