There's a small file descriptor leak in CMD_Exec() in fvwm/builtins.c;
it opens /dev/null, dup2()s it to standard input, and then never closes
the original file descriptor before exec'ing the command.
The neurotically correct fix for this (in what I think is the right
fvwm coding style) is:
---
On Tue, Jul 27, 2010 at 12:21:17PM -0400, Chris Siebenmann wrote:
> There's a small file descriptor leak in CMD_Exec() in fvwm/builtins.c;
> it opens /dev/null, dup2()s it to standard input, and then never closes
> the original file descriptor before exec'ing the command.
>
> The neurotically co
On Tue, Jul 27, 2010 at 10:49:10PM +0100, Thomas Adam wrote:
> On Tue, Jul 27, 2010 at 12:21:17PM -0400, Chris Siebenmann wrote:
> > There's a small file descriptor leak in CMD_Exec() in fvwm/builtins.c;
> > it opens /dev/null, dup2()s it to standard input, and then never closes
> > the original f