From 11e8f3ecdddf8edd8b1bbe9f41b49ce8b709b92a Mon Sep 17 00:00:00 2001 From: Petr Cech Date: Wed, 8 Jul 2015 07:17:28 -0400 Subject: UTIL: Function 2string for enum sss_cli_command MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Improvement of debug messages. Instead of:"(0x0400): Running command [17]..." We could see:"(0x0400): Running command [17][SSS_NSS_GETPWNAM]..." (It's not used in sss_client. There are only hex numbers of commands.) Resolves: https://fedorahosted.org/sssd/ticket/2708 Reviewed-by: Lukáš Slebodník --- src/providers/dp_pam_data_util.c | 27 ++------------------------- 1 file changed, 2 insertions(+), 25 deletions(-) (limited to 'src/providers') diff --git a/src/providers/dp_pam_data_util.c b/src/providers/dp_pam_data_util.c index 8724bf93..10e91f5f 100644 --- a/src/providers/dp_pam_data_util.c +++ b/src/providers/dp_pam_data_util.c @@ -23,33 +23,10 @@ */ #include "providers/data_provider.h" - +#include "util/sss_cli_cmd.h" #define PAM_SAFE_ITEM(item) item ? item : "not set" -static const char *pamcmd2str(int cmd) { - switch (cmd) { - case SSS_PAM_AUTHENTICATE: - return "PAM_AUTHENTICATE"; - case SSS_PAM_SETCRED: - return "PAM_SETCRED"; - case SSS_PAM_ACCT_MGMT: - return "PAM_ACCT_MGMT"; - case SSS_PAM_OPEN_SESSION: - return "PAM_OPEN_SESSION"; - case SSS_PAM_CLOSE_SESSION: - return "PAM_CLOSE_SESSION"; - case SSS_PAM_CHAUTHTOK: - return "PAM_CHAUTHTOK"; - case SSS_PAM_CHAUTHTOK_PRELIM: - return "PAM_CHAUTHTOK_PRELIM"; - case SSS_PAM_PREAUTH: - return "SSS_PAM_PREAUTH"; - default: - return "UNKNOWN"; - } -} - int pam_data_destructor(void *ptr) { struct pam_data *pd = talloc_get_type(ptr, struct pam_data); @@ -183,7 +160,7 @@ failed: void pam_print_data(int l, struct pam_data *pd) { - DEBUG(l, "command: %s\n", pamcmd2str(pd->cmd)); + DEBUG(l, "command: %s\n", sss_cmd2str(pd->cmd)); DEBUG(l, "domain: %s\n", PAM_SAFE_ITEM(pd->domain)); DEBUG(l, "user: %s\n", PAM_SAFE_ITEM(pd->user)); DEBUG(l, "service: %s\n", PAM_SAFE_ITEM(pd->service)); -- cgit