On Fri, 2009-01-30 at 12:28 +0200, Jonathan Quick wrote:
> (re-sending with subject)
> 
> Hi Ben
> 
> I assume you're aware of the RC bug #512839 introduced by your NMU (and in
> testing) xine-ui/0.99.5+cvs20070914-2.1 ?

No, I missed that.  The bug is clearly not grave; not sure whether it
should be RC.

> Looks like the fork()ing of xdg-screensaver needs a little more thought.

I'm afraid I was a bit lazy about this.  The previous code which forked
gnome-screensaver-command did not close fds either, so I assumed that
would be fine, but since xdg-screensaver's child processes hang around
it's not.

The correct thing to do is to close all fds above 2 before calling
execve() in the child process.

Ben.

Attachment: signature.asc
Description: This is a digitally signed message part

Reply via email to