summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorPavel Březina <pbrezina@redhat.com>2012-10-09 15:17:58 +0200
committerPavel Březina <pbrezina@redhat.com>2012-10-12 14:22:39 +0200
commit755350f1a5881a14484bc7434ec0ddc12498373f (patch)
treee5e196a5ceb40eb9849c4f0d5551daafc691b502
parent49c06899c6b8e06fa0c371dbd8a16ead32c651d9 (diff)
downloadsssd_unused-ticket-1576-increase-debugging-in-nss.zip
sssd_unused-ticket-1576-increase-debugging-in-nss.tar.gz
sssd_unused-ticket-1576-increase-debugging-in-nss.tar.xz
Add more debug information to getpwent processticket-1576-increase-debugging-in-nss
-rw-r--r--src/responder/nss/nsssrv_cmd.c43
1 files changed, 41 insertions, 2 deletions
diff --git a/src/responder/nss/nsssrv_cmd.c b/src/responder/nss/nsssrv_cmd.c
index 6848413..00eff71 100644
--- a/src/responder/nss/nsssrv_cmd.c
+++ b/src/responder/nss/nsssrv_cmd.c
@@ -1520,8 +1520,13 @@ static int nss_cmd_getpwent(struct cli_ctx *cctx)
cmdctx->saved_dom_idx = cctx->pwent_dom_idx;
cmdctx->saved_cur = cctx->pwent_cur;
+ DEBUG(0, ("dom_idx: %d, pwent_cur: %d\n",
+ cctx->pwent_dom_idx, cctx->pwent_cur));
+
nctx = talloc_get_type(cctx->rctx->pvt_ctx, struct nss_ctx);
if(!nctx->pctx || !nctx->pctx->ready) {
+ DEBUG(0, ("No data available. Calling implicit setpwent\n"));
+
/* Make sure we invoke setpwent if it hasn't been run or is still
* processing from another client
*/
@@ -1560,6 +1565,8 @@ static int nss_cmd_getpwent_immediate(struct nss_cmd_ctx *cmdctx)
return ret;
}
+ DEBUG(0, ("Calling nss_cmd_retpwen(%p, %d)\n", cctx, num));
+
ret = nss_cmd_retpwent(cctx, num);
sss_packet_set_error(cctx->creq->out, ret);
@@ -1588,14 +1595,34 @@ static int nss_cmd_retpwent(struct cli_ctx *cctx, int num)
pdom = &pctx->doms[cctx->pwent_dom_idx];
n = pdom->res->count - cctx->pwent_cur;
+
+ DEBUG(0, ("Current domain: %d Total: %d\n",
+ cctx->pwent_dom_idx, pctx->num));
+ DEBUG(0, ("Requested: %d Total: %d Current: %d Diff: %d\n",
+ num, pdom->res->count, cctx->pwent_cur, n));
+
if (n <= 0 && (cctx->pwent_dom_idx+1 < pctx->num)) {
cctx->pwent_dom_idx++;
pdom = &pctx->doms[cctx->pwent_dom_idx];
n = pdom->res->count;
cctx->pwent_cur = 0;
+
+ DEBUG(0, ("Continuing with next domain...\n"));
+ DEBUG(0, ("Current domain: %d Total: %d\n",
+ cctx->pwent_dom_idx, pctx->num));
+ DEBUG(0, ("Requested: %d Total: %d Current: %d Diff: %d\n",
+ num, pdom->res->count, cctx->pwent_cur, n));
}
- if (!n) break;
+ if (n == 0) {
+ DEBUG(0, ("OK n == 0. No more users to return. Breaking loop...\n"));
+ break;
+ }
+
+ if (n < 0) {
+ DEBUG(0, ("ERROR n < 0. Breaking loop...\n"));
+ break;
+ }
if (n > num) n = num;
@@ -1608,8 +1635,12 @@ static int nss_cmd_retpwent(struct cli_ctx *cctx, int num)
none:
if (ret == ENOENT) {
+ DEBUG(0, ("Creating empty data\n"));
ret = fill_empty(cctx->creq->out);
}
+
+ DEBUG(0, ("Returning data...\n"));
+
return ret;
}
@@ -1636,6 +1667,9 @@ static void nss_cmd_implicit_setpwent_done(struct tevent_req *req)
cmdctx->cctx->pwent_dom_idx = cmdctx->saved_dom_idx;
cmdctx->cctx->pwent_cur = cmdctx->saved_cur;
+ DEBUG(0, ("restoring dom_idx: %d, pwent_cur: %d\n",
+ cmdctx->saved_dom_idx, cmdctx->saved_cur));
+
ret = nss_cmd_getpwent_immediate(cmdctx);
if (ret != EOK) {
DEBUG(0, ("Immediate retrieval failed with unexpected error "
@@ -1659,9 +1693,14 @@ static int nss_cmd_endpwent(struct cli_ctx *cctx)
&cctx->creq->out);
if (ret != EOK) {
+ DEBUG(0, ("sss_packet_new() failed, counters are not reseted "
+ "[%d] - %s\n", ret, strerror(ret)));
return ret;
}
- if (nctx->pctx == NULL) goto done;
+ if (nctx->pctx == NULL) {
+ DEBUG(0, ("nctx->pctx == NULL, counters are not reseted\n"));
+ goto done;
+ }
/* Reset the indices so that subsequent requests start at zero */
cctx->pwent_dom_idx = 0;