On Wed, 2012-06-06 at 12:52 -0400, Simo Sorce wrote: > On Wed, 2012-06-06 at 08:19 -0400, Stephen Gallagher wrote: > > One-liner to make debugging PAM issues easier. It allows us to easily > > see what result we got back from the provider. > > Aren't you supposed to switch to the new debugging level conventions > when you touch a debug statement ? :-) > > Otherwise, ack.
I sent the wrong patch, apparently. Whoops.
From 267445a39a2cec3fee5cd95d4ff7659e492b6822 Mon Sep 17 00:00:00 2001
From: Stephen Gallagher <sgall...@redhat.com>
Date: Tue, 22 May 2012 08:39:45 -0400
Subject: [PATCH] PAM: Better pam_reply message
---
src/responder/pam/pamsrv_cmd.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/src/responder/pam/pamsrv_cmd.c b/src/responder/pam/pamsrv_cmd.c
index 608a38d4f7e3aafebe6692c6de5acb7cdf11881b..f1fdc3f0dbdd53491c58b6f26b2464a6f55b02f8 100644
--- a/src/responder/pam/pamsrv_cmd.c
+++ b/src/responder/pam/pamsrv_cmd.c
@@ -610,7 +610,8 @@ static void pam_reply(struct pam_auth_req *preq)
pctx = talloc_get_type(preq->cctx->rctx->pvt_ctx, struct pam_ctx);
- DEBUG(4, ("pam_reply get called.\n"));
+ DEBUG(SSSDBG_FUNC_DATA,
+ ("pam_reply called with result [%d].\n", pd->pam_status));
if (pd->pam_status == PAM_AUTHINFO_UNAVAIL) {
switch(pd->cmd) {
--
1.7.10.2
signature.asc
Description: This is a digitally signed message part
_______________________________________________ sssd-devel mailing list sssd-devel@lists.fedorahosted.org https://fedorahosted.org/mailman/listinfo/sssd-devel
