On Fri, Aug 30, 2024 at 01:14:50PM +0200, Clément Léger wrote: > On some systems (MacOS for instance), sysconf(_SC_OPEN_MAX) can return > -1. In that case we should fallback to using the OPEN_MAX define. > According to "man sysconf", the OPEN_MAX define should be present and > provided by either unistd.h and/or limits.h so include them for that > purpose. For other OSes, just assume a maximum of 1024 files descriptors > as a fallback. > > Fixes: 4ec5ebea078e ("qemu/osdep: Move close_all_open_fds() to oslib-posix") > Reported-by: Daniel P. Berrangé <berra...@redhat.com> > Signed-off-by: Clément Léger <cle...@rivosinc.com> > --- > util/oslib-posix.c | 12 ++++++++++++ > 1 file changed, 12 insertions(+)
Reviewed-by: Daniel P. Berrangé <berra...@redhat.com> With regards, Daniel -- |: https://berrange.com -o- https://www.flickr.com/photos/dberrange :| |: https://libvirt.org -o- https://fstop138.berrange.com :| |: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|