Your fix is a bit incomplete -- what getlogin() sometimes returns is a null string in that case -- not a NULL pointer. We also changed getlogin to return NULL if it would otherwise return "". Rick Lindsley ...{allegra,decvax,ihnp4}!tektronix!richl