On Sun, 10 Jun 2007, Paul Mackerras wrote: > > for (i = 0; i < NR_OPEN; i++) > > if (!fd_is_special_to_us(i)) > > close(i); > > > > Note that this is conceptually buggy, but occurs in several major C > > programming books, most of the major shells, and a lot of other > > software to boot. > > Buggy in what way? In the use of the NR_OPEN constant? That is typical daemonize code done by quite a few apps, and even described in many unix network programming books. Now say that before that you loaded a library that had an internal fd allocated to get some sort of services from the kernel (say an epoll fd). What happens when the library uses the cached apoll fd after that loop? - Davide - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
- References:
- Re: [patch 7/8] fdmap v2 - implement sys_socket2
- From: Ulrich Drepper <[email protected]>
- Re: [patch 7/8] fdmap v2 - implement sys_socket2
- From: Alan Cox <[email protected]>
- Re: [patch 7/8] fdmap v2 - implement sys_socket2
- From: Kyle Moffett <[email protected]>
- Re: [patch 7/8] fdmap v2 - implement sys_socket2
- From: Eric Dumazet <[email protected]>
- Re: [patch 7/8] fdmap v2 - implement sys_socket2
- From: Davide Libenzi <[email protected]>
- Re: [patch 7/8] fdmap v2 - implement sys_socket2
- From: Eric Dumazet <[email protected]>
- Re: [patch 7/8] fdmap v2 - implement sys_socket2
- From: Ulrich Drepper <[email protected]>
- Re: [patch 7/8] fdmap v2 - implement sys_socket2
- From: Davide Libenzi <[email protected]>
- Re: [patch 7/8] fdmap v2 - implement sys_socket2
- From: Ulrich Drepper <[email protected]>
- Re: [patch 7/8] fdmap v2 - implement sys_socket2
- From: Davide Libenzi <[email protected]>
- Re: [patch 7/8] fdmap v2 - implement sys_socket2
- From: Theodore Tso <[email protected]>
- Re: [patch 7/8] fdmap v2 - implement sys_socket2
- From: Alan Cox <[email protected]>
- Re: [patch 7/8] fdmap v2 - implement sys_socket2
- From: Davide Libenzi <[email protected]>
- Re: [patch 7/8] fdmap v2 - implement sys_socket2
- From: Paul Mackerras <[email protected]>
- Re: [patch 7/8] fdmap v2 - implement sys_socket2
- From: Kyle Moffett <[email protected]>
- Re: [patch 7/8] fdmap v2 - implement sys_socket2
- From: Paul Mackerras <[email protected]>
- Re: [patch 7/8] fdmap v2 - implement sys_socket2
- Prev by Date: Re: [patch 7/8] fdmap v2 - implement sys_socket2
- Next by Date: [BUG] fs/buffer.c:1821 in 2.6.22-rc4-mm2
- Previous by thread: Re: [patch 7/8] fdmap v2 - implement sys_socket2
- Next by thread: Re: [patch 7/8] fdmap v2 - implement sys_socket2
- Index(es):