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.