Matthew,

Sorry for the delay in replying.

I believe your patch is correct.

I will submit the patch to MrSam for inclusion in the courier-imap
distribution.

Michael.


> Matthew Kanar wrote:
>
> > I found that changing cdb passwords through courier-imap was failing for
> > others and me with the courier-imap developement release,
> > courier-imap-2.2.2.20040207.  This patch seems to correct the issue.
> > The change is very simple.  "vpw->pw_passwd" in authvchkpw.c is changed
> > to "vpw->pw_passwd == 0".  Somebody else, most notably Michael Bowe
> > <[EMAIL PROTECTED]>, should probably review the change.
> >
> > Thank You,
> > Matthew Kanar
> >
> > 2004-03-02  Matthew Kanar <[EMAIL PROTECTED]>
> >
> >        * authlib/authvchkpw.c: Made a change so that vpopmail
> >                                password changes work again.
> >
> > --- ../courier-imap-2.2.2.20040207/authlib/authvchkpw.c 2004-01-14
> > 22:17:01.000000000 -0500
> > +++ authlib/authvchkpw.c        2004-03-02 12:52:52.136420151 -0500
> > @@ -190,7 +190,7 @@
> >         *   - user's password field in the passwd entry is empty
> >         *   - supplied current password doesnt match stored password
> >         */
> > -       if (vpw->pw_passwd || authcheckpassword(pass, vpw->pw_passwd)) {
> > +       if (vpw->pw_passwd == 0 || authcheckpassword(pass,
> > vpw->pw_passwd)) {
> >                errno=EPERM;
> >                return (-1);
> >        }
>

Reply via email to