add closefrom() call
Joerg Sonnenberger
joerg at britannica.bec.de
Fri Jul 6 12:42:13 UTC 2007
On Fri, Jul 06, 2007 at 06:18:14PM +0800, LI Xin wrote:
> - closefrom(-1) would be no-op on DragonFly, my version would close all
> open files (From my understanding of OpenSolaris's userland
> implementation, this is Solaris's behavior).
I think this is a bad idea as -1 is generally an invalid file
descriptor.
> - my version closefrom(very_big_fd) would result in EBADF. I am not
> very sure whether this is correct, but it does not hurt for applications
> that thinks closefrom() would return void.
I don't think this is a good idea either. One typical use is closefrom(3)
and returning an error because no such descriptors are open sounds very
wrong. It also just adds another special case as the loop handles this
already....
Joerg
More information about the freebsd-hackers
mailing list