Bug 4073 - select() doesn't use highestfd as first argument
Status: RESOLVED FIXED
Alias: None
Product: ioquake3
Classification: Unclassified
Component: Misc
Version: unspecified
Hardware: PC Linux
: P3 normal
Assignee: Zachary J. Slater
QA Contact: ioquake3 bugzilla mailing list
URL:
Depends on:
Blocks:
 
Reported: 2009-06-05 13:14 EDT by Ben Millwood
Modified: 2009-06-05 14:55:28 EDT
1 user (show)

See Also:

Attachments
nfds set to highestfd + 1 (721 bytes, patch)
2009-06-05 13:14 EDT, Ben Millwood

Description Ben Millwood 2009-06-05 13:14:52 EDT 
Created attachment 2073 [details]
nfds set to highestfd + 1

This one's pretty straightforward. The first argument of select() is supposed to be the max file descriptor being watched + 1. To this end, int highestfd is declared and set, but the actual select() call doesn't use it.

git format-patch attached.
Comment 1 Thilo Schulz 2009-06-05 14:55:28 EDT 
How in the world did I miss that?

Thanks. Fixed.