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); > > } >