From 605499bfc2ca99aa63a9b0aded82dd990cf12eaa Mon Sep 17 00:00:00 2001 From: Bruce Mitchener Date: Tue, 15 Dec 2020 02:55:08 +0700 Subject: [PATCH] Remove getdtablesize calls in favor of sysconf(_SC_OPEN_MAX). (#45) `getdtablesize()` has long been deprecated in favor of `sysconf(_SC_OPEN_MAX)`. --- src/unixcomm.c | 4 ---- src/unixfork.c | 7 +------ 2 files changed, 1 insertion(+), 10 deletions(-) diff --git a/src/unixcomm.c b/src/unixcomm.c index def34e6..4edcbee 100644 --- a/src/unixcomm.c +++ b/src/unixcomm.c @@ -277,11 +277,7 @@ int FindUnixPipes(void) { if ((envtmp = getenv("LDEUNIXPID"))) UnixPID = atoi(envtmp); /* This is a good place to initialize stuff like the UJ table */ -#ifdef SYSVONLY NPROCS = sysconf(_SC_OPEN_MAX); -#else - NPROCS = getdtablesize(); -#endif /* SYSVONLY */ UJ = (struct unixjob *)malloc(NPROCS * sizeof(struct unixjob)); cleareduj.status = -1; diff --git a/src/unixfork.c b/src/unixfork.c index c38c2db..09c0c09 100644 --- a/src/unixfork.c +++ b/src/unixfork.c @@ -446,13 +446,8 @@ int fork_Unix() { dup2(sock, 1); dup2(sock, 2); -#ifdef SYSVONLY - /* Make sure everything else is closed POSIX has no getdtab... */ + /* Make sure everything else is closed. */ for (i = 3; i < sysconf(_SC_OPEN_MAX); i++) close(i); -#else - /* Make sure everything else is closed */ - for (i = 3; i < getdtablesize(); i++) close(i); -#endif /* SYSVONLY */ /* Run the shell command and get the result */ status = system(cmdstring);