Bug 5764 - Xjewel uses 100% CPU when started from window manager
Xjewel uses 100% CPU when started from window manager
Status: CLOSED RAWHIDE
Product: Red Hat Linux
Classification: Retired
Component: xjewel (Show other bugs)
6.0
i386 Linux
medium Severity medium
: ---
: ---
Assigned To: Michael K. Johnson
:
Depends On:
Blocks:
  Show dependency treegraph
 
Reported: 1999-10-09 07:37 EDT by k.h.c.vanhouten
Modified: 2008-05-01 11:37 EDT (History)
1 user (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2000-01-13 17:44:09 EST
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---


Attachments (Terms of Use)

  None (edit)
Description k.h.c.vanhouten 1999-10-09 07:37:54 EDT
When xjewel is started with it's stdio redirected from and
to /dev/null, it will consume 100% cpu. This is the result
of incorrect arguments to select().
To demonstrate:
xjewel < /dev/null > /dev/null 2>&1 &

(Now watch CPU with xsoview).

I have a patch for this problem.
Comment 1 Michael K. Johnson 1999-11-22 13:54:59 EST
Thank you.  You say you have a patch; could you please use the
"Create a new attachment" feature of bugzilla (you'll see it
under the Summary line) to include it?
Comment 2 k.h.c.vanhouten 1999-11-23 12:31:59 EST
I couldn't find "Create a new attachment", But here is the patch:

--- xjewel-1.6.orig/xw.c        Wed Feb 16 11:34:51 1994
+++ xjewel-1.6/xw.c     Sun Oct  3 15:17:30 1999
@@ -156,6 +156,8 @@
        fd_set readfds, writefds, exceptfds;
        struct timeval timeout_BSD;

+       FD_ZERO(&readfds);
+       FD_ZERO(&exceptfds);
        FD_SET(XConnectionNumber(xw_display),&readfds);
        FD_SET(XConnectionNumber(xw_display),&exceptfds);
 #   else
@@ -224,6 +226,8 @@
                        }
                else
                        { ret = select(nfds,&readfds,NULL,&exceptfds,NULL); }
+               FD_ZERO(&readfds);
+               FD_ZERO(&exceptfds);
                FD_SET(XConnectionNumber(xw_display),&readfds);
                FD_SET(XConnectionNumber(xw_display),&exceptfds);
 #else
Comment 3 Nalin Dahyabhai 2000-01-07 13:14:59 EST
Thanks for the patch, it's been applied.  Please verify it with
xjewel-1.6-12, which should be in the next Raw Hide.

Note You need to log in before you can comment on or make changes to this bug.