Launchpad has imported 5 comments from the remote bug at
https://bugzilla.redhat.com/show_bug.cgi?id=481146.

If you reply to an imported comment from within Launchpad, your comment
will be sent to the remote bug automatically. Read more about
Launchpad's inter-bugtracker facilities at
https://help.launchpad.net/InterBugTracking.

------------------------------------------------------------------------
On 2009-01-22T12:58:44+00:00 John wrote:

Created attachment 329709
proposed fix

Description of problem:

When (without -pipe) the child exits, state->pipe is cleared but
state->pid is not. Later on, state->pid is used to determine whether to
send a message to the child (via state->pipe) it tests only the value of
state->pid.

This can be forced by running with -window and resizing the window
between child runs (which attempts to ioctl TIOCWINSZ on the pipe).

Version-Release number of selected component (if applicable):

xscreensaver-extras-5.08-1.fc9.x86_64

How reproducible:

The resize case, very. I think there's another semi-random crash case
too though.

Steps to Reproduce:
1. Run phosphor with -window
2. Resize window
3.
  
Actual results:

SEGV

Expected results:

No SEGV

Additional info:

Attached patch does two things: clear state->pid once we know the child
has exited, and check state->pipe before attempting to dereference it.

Reply at:
https://bugs.launchpad.net/ubuntu/+source/xscreensaver/+bug/357556/comments/0

------------------------------------------------------------------------
On 2009-01-22T13:54:40+00:00 Mamoru wrote:

Well, I can reproduce what you see and your proposal patch
seems reasonable. I will release the modified xscreensaver,
thanks.

Reply at:
https://bugs.launchpad.net/ubuntu/+source/xscreensaver/+bug/357556/comments/1

------------------------------------------------------------------------
On 2009-01-22T15:45:11+00:00 Mamoru wrote:

5.08-5.fc{11,10,9} is submitted:
For F-9:
http://koji.fedoraproject.org/koji/buildinfo?buildID=79778

Reply at:
https://bugs.launchpad.net/ubuntu/+source/xscreensaver/+bug/357556/comments/2

------------------------------------------------------------------------
On 2009-01-24T02:39:42+00:00 Fedora wrote:

xscreensaver-5.08-5.fc10 has been pushed to the Fedora 10 stable
repository.  If problems still persist, please make note of it in this
bug report.

Reply at:
https://bugs.launchpad.net/ubuntu/+source/xscreensaver/+bug/357556/comments/3

------------------------------------------------------------------------
On 2009-01-24T02:43:10+00:00 Fedora wrote:

xscreensaver-5.08-5.fc9 has been pushed to the Fedora 9 stable
repository.  If problems still persist, please make note of it in this
bug report.

Reply at:
https://bugs.launchpad.net/ubuntu/+source/xscreensaver/+bug/357556/comments/4


** Changed in: xscreensaver (Fedora)
   Importance: Unknown => Medium

-- 
You received this bug notification because you are a member of Desktop
Packages, which is subscribed to xscreensaver in Ubuntu.
https://bugs.launchpad.net/bugs/357556

Title:
  phosphor crashed with SIGSEGV in fileno_unlocked()

Status in xscreensaver package in Ubuntu:
  Fix Released
Status in xscreensaver package in Debian:
  Fix Released
Status in xscreensaver package in Fedora:
  Fix Released

Bug description:
  Binary package hint: xscreensaver

  This happens with all the computers that I have, both under Intrepid
  and under Jaunty.  The symptom is that after the screensaver has been
  running for a short time (between 30 seconds and half an hour), the
  screen will go black and no longer run the screensaver.  Unlocking the
  screen and re-locking the screen (to restart the screensaver) causes
  the screensaver to display normally for the same period of time, and
  then once again crash and go back to a black screen.

  A minimal patch has been created to fix the problem.  Since this bug
  makes the Phosphor screensaver unusable as-is, and this is not a core
  component, I am requesting a stable version update for this package.

  The worst case scenario is that this patch causes a regression in
  Phosphor, which does not work as is, so the risk is minimal.

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/xscreensaver/+bug/357556/+subscriptions

-- 
Mailing list: https://launchpad.net/~desktop-packages
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~desktop-packages
More help   : https://help.launchpad.net/ListHelp

Reply via email to