* Re: [PATCH] sys_poll SuS compliance fix
[not found] <200212181631.gBIGVLJ11794@hera.kernel.org>
@ 2002-12-18 17:06 ` Jeff Garzik
0 siblings, 0 replies; only message in thread
From: Jeff Garzik @ 2002-12-18 17:06 UTC (permalink / raw)
To: Linux Kernel Mailing List, manfred
Linux Kernel Mailing List wrote:
> ChangeSet 1.942, 2002/12/18 10:33:48-02:00, manfred@colorfullife.com
>
> [PATCH] sys_poll SuS compliance fix
> diff -Nru a/fs/select.c b/fs/select.c
> --- a/fs/select.c Wed Dec 18 08:31:22 2002
> +++ b/fs/select.c Wed Dec 18 08:31:22 2002
> @@ -417,7 +417,7 @@
> int nchunks, nleft;
>
> /* Do a sanity check on nfds ... */
> - if (nfds > NR_OPEN)
> + if (nfds > current->files->max_fdset && nfds > OPEN_MAX)
> return -EINVAL;
The changeset description is awful, can you give us more details
Manfred? [also in the future can you please give Linus more description
with your patches?]
In particular, I wonder if "||" is more appropriate than "&&"?
Jeff
^ permalink raw reply [flat|nested] only message in thread
only message in thread, other threads:[~2002-12-18 16:58 UTC | newest]
Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <200212181631.gBIGVLJ11794@hera.kernel.org>
2002-12-18 17:06 ` [PATCH] sys_poll SuS compliance fix Jeff Garzik
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.