gark...@mailueberfall.de wrote > I understand that you don't agree with the bug being grave. The point is > that I use it mainly for watching TV. > This is practically not possible anymore now since the VDR playback > breaks on every channel switch due to xdg-screensaver hogging the audio > handle. For that usage it is definately a grave bug because it rendered > my main application useless. > I know this may not affect all users, but I also find it a bit harsh to > say a bug is clearly not grave just because one is mainly using a xine > plugin that has been effectively broken now.
I sympathise, but the definition is:
grave
makes the package in question unusable or mostly so, or causes
data loss, or introduces a security hole allowing access to the
accounts of users who use the package.
I think that this bug is:
important
a bug which has a major effect on the usability of a package,
without rendering it completely unusable to everyone.
but it's up to the maintainer to decide.
> Just as a remark regarding your intended fix: I haven't yet seen the
> place where xine does a wait() or waitpid() for those forked childs to
> clean up the zombies. If there is a place, ok, I just havn't seen it
> yet.
I added signal(SIGCHLD, SIG_IGN) which should make that unnecessary.
Also, the change I actually made to xine-ui *does* include a loop to
close fds 3-255 (sadly there is no easy way to find out what the last
open fd is). So unless xine is opening a *huge* number of files I can't
understand this bug.
Please could you run "ls -l /proc/$pid/fd/" for the pid of the
xdg-screensaver process when this bug is triggered?
> If I startup the XVDR plugin, I see three pid's of xdg-screensaver. Upon
> each TV channel switch, there's one more. They also don't seem to
> terminate, so the list is growing. Aren't they supposed to just signal a
> "suspend" and then terminate themselves?
Which version of xdg-utils do you have installed? There was a bug in
that which could result in zombie processes, but I fixed that before
making this change.
Ben.
signature.asc
Description: This is a digitally signed message part
