Package: xdm Version: 4.3.0.dfsg.1-7 Severity: normal Note to self: set submitter to Chip Coldwell <[EMAIL PROTECTED]> once an ack is recived for this bug.
I don't think there are any security implications to this bug, since
unprivileged users cannot kill the (root-owned) Xwilling process (which is
a script that lives in /etc/X11 on Debian, and is not writable by users).
Since the only people who could use this vector to DoS xdm have root
privileges anyway, this bug merits severity "normal".
For a more direct method of DoSsing xdm as root, try:
kill -STOP $(pidof xdm)
:-/
----- Forwarded message from Chip Coldwell <[EMAIL PROTECTED]> -----
From: Chip Coldwell <[EMAIL PROTECTED]>
To: Branden Robinson <[EMAIL PROTECTED]>
Cc: Debian X Strike Force <debian-x@lists.debian.org>
Subject: xdm race condition
Date: Thu, 24 Jun 2004 13:58:06 -0400 (EDT)
Message-ID: <[EMAIL PROTECTED]>
X-Mailing-List: <debian-x@lists.debian.org> archive/latest/19682
X-Spam-Status: No, hits=-5.0 required=4.0 tests=LDOSUBSCRIBER autolearn=no
version=2.63-lists.debian.org_2004_06_20_05
I found another xdm bug. This time it's a race condition in
xc/programs/xdm/policy.c:Willing
around line numbers 140--145, which reads
if ((fd = popen(willing, "r")))
{
char *s = NULL;
while(!(s = fgets(statusBuf, 256, fd)) && errno == EINTR)
;
Here's the problem. The "popen" call creates a child process and a
pipe to communicate with it. If the child process exits during the
"fgets" call without generating any output, the parent process
receives SIGCHLD and the read system call gets interrupted. Therefore
errno == EINTR, and since the child has exited the pipe never returns
any data. xdm goes into an infinite loop. I think the problem is
that fgets doesn't reset errno to zero; we have to do that manually.
The fix is the trivial patch attached to this email.
(The child process is the "Xwilling" script; in the case of the default
Debian configuration it is
"su nobody -c /usr/X11R6/lib/X11/xdm/Xwilling")
Chip
--
Charles M. "Chip" Coldwell
System Administrator
Harvard Physics Department
617-495-3388
Content-Description: xdm race condition fix
--- xc/programs/xdm/policy.c~ 2002-12-07 15:31:04.000000000 -0500
+++ xc/programs/xdm/policy.c 2004-06-24 09:56:19.000000000 -0400
@@ -140,8 +140,9 @@
if ((fd = popen(willing, "r")))
{
char *s = NULL;
+ errno = 0;
while(!(s = fgets(statusBuf, 256, fd)) && errno == EINTR)
- ;
+ errno = 0;
if (s && strlen(statusBuf) > 0)
statusBuf[strlen(statusBuf)-1] = 0; /* chop newline */
else
----- End forwarded message -----
--
G. Branden Robinson | Life is what happens to you while
Debian GNU/Linux | you're busy making other plans.
[EMAIL PROTECTED] | -- John Lennon
http://people.debian.org/~branden/ |
signature.asc
Description: Digital signature
