On Fri, Feb 27, 2015 at 10:21:20AM +0100, Jakub Hrozek wrote:
On Thu, Feb 26, 2015 at 10:43:53AM +0100, Pavel Reichl wrote:
> >>--- a/src/responder/pam/pamsrv_cmd.c
> >>+++ b/src/responder/pam/pamsrv_cmd.c
> >>@@ -679,6 +679,12 @@ static void pam_reply(struct pam_auth_req *preq)
> >> ret = confdb_get_string(pctx->rctx->cdb, pd,
CONFDB_PAM_CONF_ENTRY,
> >> CONFDB_PAM_ACCOUNT_EXPIRED_MESSAGE,
"",
> >> &pam_account_expired_message);
> >>+ if (ret != EOK) {
> >>+ DEBUG(SSSDBG_MINOR_FAILURE,
> >>+ "Failed to parse expiration message:
%d:[%s].\n",
> >Are you sure the message should say parse and not get or retrieve?
> No, I agree with you that get is a better choice.
>
> Please see updated patch.
>
> Thanks!
>
> ci:
http://sssd-ci.idm.lab.eng.brq.redhat.com:8080/job/ci/832/
ACK
* master: c5290f2175845f2c5e3f35ce279b6f52b1d51275
* sssd-1-12: 45b7cadbbd81df4a6aba39de97eb1522774bcc60