Breakage in crypt.c

Started by Tom Lanealmost 24 years ago2 messageshackers
Jump to latest
#1Tom Lane
tgl@sss.pgh.pa.us

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

#2Bruce Momjian
bruce@momjian.us
In reply to: Tom Lane (#1)
Re: Breakage in crypt.c

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, Pennsylvania 19026