Thread: Breakage in crypt.c

There are various paths of control in md5_crypt_verify that do
    if (passwd)        pfree(passwd);    if (valuntil)        pfree(valuntil);

Isn't this now pfree'ing part of the saved pre-parsed pg_pwd data?
        regards, tom lane

Tom Lane wrote:
> There are various paths of control in md5_crypt_verify that do
> 
>         if (passwd)
>             pfree(passwd);
>         if (valuntil)
>             pfree(valuntil);
> 
> Isn't this now pfree'ing part of the saved pre-parsed pg_pwd data?

Oops, yep.  Fixed.  The pfree's were fine in 7.2, but now, we cache
pg_pwd.

--  Bruce Momjian                        |  http://candle.pha.pa.us pgman@candle.pha.pa.us               |  (610)
853-3000+  If your life is a hard drive,     |  830 Blythe Avenue +  Christ can be your backup.        |  Drexel Hill,
Pennsylvania19026