On Fri, Aug 01, 2008 at 11:08:34AM +0200, Julien BLACHE wrote: > Nick Andrew <nick at nick-andrew.net> wrote: > > When /dev/null is opened it gets the lowest numbered unopened > > file descriptor. If any of the file descriptors 0, 1 or 2 are > > closed when the program starts running, the later close() > > will cause that file descriptor to be closed, which is not > > wanted. > > There is no reason why this would happen, so unless you've seen that > actually happening...
It's just basic defensive programming. The intent of saned is obviously to sanitise file descriptors 0, 1 and 2 by opening them all to /dev/null; if any of those file descriptors weren't open at program start then the sanitising will fail because one of the file descriptors will end up closed by accident. My patch merely ensures that the code delivers what was intended by the author. Nick. -- PGP Key ID = 0x418487E7 http://www.nick-andrew.net/ PGP Key fingerprint = B3ED 6894 8E49 1770 C24A 67E3 6266 6EB9 4184 87E7
