On 09/18/2015 05:07 PM, Michal Židek wrote:
+static errno_t check_cert(struct cli_ctx *cctx, + struct pam_ctx *pctx, + struct tevent_req *req,
What's the point of parameter *req? You are not reading it and if you want to use it as output parameter you should use **_req, right?
+ struct pam_auth_req *preq, + struct pam_data *pd) +{ + int p11_child_timeout; + const int P11_CHILD_TIMEOUT_DEFAULT = 10; + errno_t ret; + + ret = confdb_get_int(pctx->rctx->cdb, CONFDB_PAM_CONF_ENTRY, + CONFDB_PAM_P11_CHILD_TIMEOUT, + P11_CHILD_TIMEOUT_DEFAULT, + &p11_child_timeout); + if (ret != EOK) { + DEBUG(SSSDBG_CRIT_FAILURE, + "Failed to read p11_child_timeout from confdb: [%d]: %s\n", + ret, sss_strerror(ret)); + return ret; + } + + req = pam_check_cert_send(cctx, cctx->ev, pctx->p11_child_debug_fd, + pctx->nss_db, p11_child_timeout, pd); + if (req == NULL) { + DEBUG(SSSDBG_OP_FAILURE, "pam_check_cert_send failed.\n"); + return ENOMEM; + } else { + tevent_req_set_callback(req, pam_forwarder_cert_cb, preq); + return EAGAIN; + } + + return EOK; + +}
_______________________________________________ sssd-devel mailing list sssd-devel@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/sssd-devel