xdm: Changes to 'master'

[Julien Cristau]

 auth.c       |   85 ++++++++++++++++++++++++++++++++++++++++------
 configure.ac |   17 +++++++++
 dm.c         |  108 ++++++++++++++++++++++++++++++++---------------------------
 session.c    |   89 +++++++++++++++++++++++++++++++++++++++++-------
 4 files changed, 226 insertions(+), 73 deletions(-)

New commits:
commit c925709069d7515bb94f43e4b88155a95e41c8fe
Author: Branden Robinson <branden at debian.org>
Date:   Mon Nov 18 04:24:48 2002 -0500

    rewrite StorePid()
    
    Update StorePid() function to use Matthieu Herrb's changes that refuse
    to open an existing pid file, and take more care with the size of the
    Pid_t type.
    
    Matthieu Herrb rewrote the StorePid() function based on my changes; he
    made it more careful with the size of Pid_t, and treats an existing pid
    file as a failure.  I then made some more changes: use _SysErrorMsg()
    instead of strerror(), and don't assume that the process id we write to
    the pidFile is 5 digits.
    
    Forward-ported by David Nusinow, Eugene Konev and Julien Cristau.
    
    Signed-off-by: Julien Cristau <jcristau at debian.org>
    Reviewed-by: Alan Coopersmith <alan.coopersmith at sun.com>
    Reviewed-by: Matthieu Herrb <matthieu.herrb at laas.fr>

commit 6d393844dca10823f85d1ac797879fc6e00eae59
Author: Manoj Srivastava <srivasta at golden-gryphon.com>
Date:   Tue Jan 5 16:40:21 2010 +0000

    xdm: add SELinux support
    
    Initial patch submitted in Debian bug#233551.
    Forward-ported to modular X by Eugene Konev (changes: remove Imakefile hunks,
    add --with-selinux flag to configure.ac).
    Updated to latest SE Linux code by Russell Coker 3rd Aug 2008, with bugfix from
    Julien Cristau (Debian bug#493524).
    
    Signed-off-by: Julien Cristau <jcristau at debian.org>
    Reviewed-by: Alan Coopersmith <alan.coopersmith at sun.com>
    Reviewed-by: Matthieu Herrb <matthieu.herrb at laas.fr>

commit 90c2cc57cdf911cec2eec185f357868209212c79
Author: Branden Robinson <branden at debian.org>
Date:   Tue Jan 5 16:40:21 2010 +0000

    Improve logging of xdm's script handling
    
    * Make execution of the session program an informational log message (with
      LogInfo()), not a Debug() message.
    * Ensure that source()d scripts (Xreset, Xsetup, Xstartup) can be
      fopen()ed before invoking runAndWait() on them.  If they can, LogInfo()
      the fact.  If they cannot, LogInfo() that, too, and use _SysErrorMsg() to
      report why fopen() failed.
    * Make source() complain using Debug() when it is given a null pointer in
      its file argument.
    * For clarity, make source() return from only one location.
    
    Also see Debian bug #219311.
    
    Forward ported by Eugene Konev.
    
    Signed-off-by: Julien Cristau <jcristau at debian.org>
    Reviewed-by: Alan Coopersmith <alan.coopersmith at sun.com>
    Reviewed-by: Matthieu Herrb <matthieu.herrb at laas.fr>

commit 5222d28e8d8e5b4cc705f1a637aace405ad32bd5
Author: Branden Robinson <branden at debian.org>
Date:   Tue Jan 5 16:40:21 2010 +0000

    improve error logging
    
    Make several LogError() and Debug() messages more informative.
    Change LogError() invocations to use _SysErrorMsg() where errno might be set
    (and not clobbered by intermediate calls).  Also make LogError() the first
    thing we do after an error condition in those cases.
    
    Forward-ported by Julien Cristau <jcristau at debian.org>.
    
    Signed-off-by: Julien Cristau <jcristau at debian.org>
    Reviewed-by: Alan Coopersmith <alan.coopersmith at sun.com>
    Reviewed-by: Matthieu Herrb <matthieu.herrb at laas.fr>

commit 17105faffcb4533da71f0d3d17fd40a64b6faf50
Author: Branden Robinson <branden at debian.org>
Date:   Tue Jan 5 16:40:21 2010 +0000

    Always attempt to write data to the auth file
    
    Actually attempt to write data to the auth file in SaveServerAuthorizations();
    that way we detect problems like ENOSPC (full filesystem, user over quota)
    early enough to do something about it.  (See Debian #217505, #253480)
    
    Forward-ported by Julien Cristau <jcristau at debian.org>.
    
    Signed-off-by: Julien Cristau <jcristau at debian.org>
    Reviewed-by: Alan Coopersmith <alan.coopersmith at sun.com>
    Reviewed-by: Matthieu Herrb <matthieu.herrb at laas.fr>

commit 11d6aa0b392daef2900681b70960d8be4ada89a9
Author: Branden Robinson <branden at debian.org>
Date:   Tue Jan 5 16:40:21 2010 +0000

    Change openFiles() to avoid race-based symlink attacks.
    
    Forward-ported by Julien Cristau <jcristau at debian.org>.
    
    Signed-off-by: Julien Cristau <jcristau at debian.org>
    Reviewed-by: Alan Coopersmith <alan.coopersmith at sun.com>
    Reviewed-by: Matthieu Herrb <matthieu.herrb at laas.fr>

commit cd6b6d5f78e92725d6466d138f5441d791e58212
Author: Branden Robinson <branden at debian.org>
Date:   Tue Jan 5 16:40:21 2010 +0000

    downgrade an error to info message
    
    During normal xdm shutdown, killed local X servers
    can be zombies; this is not an error.
    
    Forward-ported by David Nusinow, Eugene Konev and Julien Cristau
    
    Signed-off-by: Julien Cristau <jcristau at debian.org>
    Reviewed-by: Alan Coopersmith <alan.coopersmith at sun.com>
    Reviewed-by: Matthieu Herrb <matthieu.herrb at laas.fr>