Does this make sense:

char *  
crypt_getpwdfilename()
{ 
    
  static char *pfnam = NULL;
  
  if (!pfnam)
  { 
    int bufsize;
    bufsize = strlen(DataDir) + strlen(CRYPT_PWD_FILE) + 2;
    pfnam = (char *) palloc(bufsize); 
    spprintf(pfnam, bufsize, "%s/%s", DataDir, CRYPT_PWD_FILE);
  }
  
  return pfnam;
}

Why the check for '!ipfnam'?  Seems useless since we are setting it to
NULL the line before...no?

Marc G. Fournier                                
Systems Administrator @ hub.org 
primary: scrappy@hub.org           secondary: scrappy@{freebsd|postgresql}.org