# include "config.h"
#endif
-#include <vlc_common.h>
+#ifdef HAVE_POLL
+#include <stdlib.h>
+
+struct pollfd;
-#ifndef HAVE_POLL
+int vlc_poll (struct pollfd *fds, unsigned nfds, int timeout)
+{
+ (void)fds; (void)nfds; (void)timeout;
+ abort ();
+}
+#else /* !HAVE_POLL */
+
+#include <vlc_common.h>
#include <string.h>
#include <stdlib.h>
#include <vlc_network.h>
struct timeval tv = { 0, 0 };
int val = -1;
-
FD_ZERO (&rdset);
FD_ZERO (&wrset);
FD_ZERO (&exset);
if (val < fd)
val = fd;
- /* I assume the OS has a solution select overflow if it does not have
- * poll(). If it did not, we are screwed anyway. */
+ /* With POSIX, FD_SET & FD_ISSET are not defined if fd is negative or
+ * bigger or equal than FD_SETSIZE. That is one of the reasons why VLC
+ * uses poll() rather than select(). Most POSIX systems implement
+ * fd_set has a bit field with no sanity checks. This is especially bad
+ * on systems (such as BSD) that have no process open files limit by
+ * default, such that it is quite feasible to get fd >= FD_SETSIZE.
+ * The next instructions will result in a buffer overflow if run on
+ * a POSIX system, and the later FD_ISSET will do undefined memory
+ * access.
+ *
+ * With Winsock, fd_set is a table of integers. This is awfully slow.
+ * However, FD_SET and FD_ISSET silently and safely discard
+ * overflows. If it happens we will loose socket events. Note that
+ * most (if not all) Winsock SOCKET handles are actually bigger than
+ * FD_SETSIZE in terms of absolute value - they are not POSIX file
+ * descriptors. From Vista, there is a much nicer WSAPoll(), but Mingw
+ * is yet to support it.
+ *
+ * With BeOS, the situation is unknown (FIXME: document).
+ */
if (fds[i].events & POLLIN)
FD_SET (fd, &rdset);
if (fds[i].events & POLLOUT)
FD_SET (fd, &exset);
}
+#ifndef HAVE_ALERTABLE_SELECT
+# warning FIXME! Fix cancellation and remove this crap.
+resume:
+ vlc_testcancel ();
+
+ if ((timeout < 0) || (timeout > 50))
+ {
+ tv.tv_sec = 0;
+ tv.tv_usec = 50;
+ }
+ else
+#endif
if (timeout >= 0)
{
div_t d = div (timeout, 1000);
val = select (val + 1, &rdset, &wrset, &exset,
(timeout >= 0) ? &tv : NULL);
+
+#ifndef HAVE_ALERTABLE_SELECT
+# warning FIXME! Fix cancellation and remove this crap.
+ if (val == 0)
+ {
+ if (timeout > 0)
+ timeout -= (timeout > 50) ? 50 : timeout;
+ if (timeout != 0)
+ goto resume;
+ }
+#endif
+
if (val == -1)
return -1;