Skip to contentSkip to conversation

sys_poll SuS compliance fix

PATCH Jeff Garzik 1 participant 1 message Dec 18, 2002 1 message not in the archive

lore
Share & tools
lore

A message that is not in the archive (sent off-list or not yet received): [email protected]

#1 lore raw
Quoted 8 lines · Linux Kernel Mailing List

ChangeSet 1.942, 2002/12/18 10:33:48-02:00, [email protected]

[PATCH] sys_poll SuS compliance fix

diff -Nru a/fs/select.c b/fs/select.c

fs/select.c line 421

Quoted 6 lines
fs/select.c +1 -1
@@ -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