tag 206141 + patch
thanks

On Tue, Aug 19, 2003 at 09:49:31AM +1000, Neil Brown wrote:
> When xdm cannot write to $HOME/.Xauthority, such as when the user is over 
> quota,
> it tried to create a .Xauthority file in /tmp and use that.
> This breaks due to patch/002_xdm_fixes.diff
> In particular openFiles causes a segmentation violation.
> 
> Towards the end of the patches openFile are the lines:
> 
> +     if (oldp)
> +         *oldp = fopen (name, "r");
> +     if (*oldp)
> +         Debug ("open succeeded %s\n", name);
> 
> if oldp is NULL, as it will be in this case, the
>    if (*oldp)
> will dereference a NULL pointer - and BANG.

Thanks very much for this bug report and analysis.  I wish more bug
reports against my packages were like yours.

-- 
G. Branden Robinson                |    America is at that awkward stage.
Debian GNU/Linux                   |    It's too late to work within the
[EMAIL PROTECTED]                 |    system, but too early to shoot the
http://people.debian.org/~branden/ |    bastards.           -- Claire Wolfe

Attachment: pgpBl63vN2pXA.pgp
Description: PGP signature

Reply via email to