Re: [libvirt PATCH 2/2] tests: don't set G_DEBUG=fatal-warnings on macOS

On Thu, Apr 28, 2022 at 02:52:45PM +0100, Daniel P. Berrangé wrote: > > #else > > if (select() < 0) > > if (!EBADF) > > return -1; > > #endif > > > > > instead? If acting on an fd that's already been closed is okay when > > using the poll()-based implementation, the same should apply to the > > select()-based one as well. > > The only way to get the same semantics for select() would be to see > EBADF and then iterate over struct pollfd, calling fcntl() to see > which one(s) are BADF. Possible, but its unpleasantly non-scalable, > so I imagine that's why they didn't do this. IIUC g_poll(), just like any other function used as GPollFunc, is supposed to have the same semantics as poll()[1], which means that passing an invalid FD should result in the corresponding flag being set rather than an outright failure. In other words, the current implementation of g_poll() on macOS doesn't follow the contract defined by GLib itself. It seems to me that this is a (fairly serious) bug in the library, no?