FIX: force use of setenv function instead of putenv on Posix targets

(as `putenv` is causing memory leaks)
Oldes · Jul 27, 2020 · 8d24585 · 8d24585
1 parent 5a3cbe5
commit 8d24585
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/src/include/reb-config.h b/src/include/reb-config.h
@@ -188,10 +188,12 @@ These are now obsolete (as of A107) and should be removed:
 
 #ifdef TO_ANY_LINUX
 #undef USE_MIDI_DEVICE // Not implemented!
+#define USE_SETENV 
 #endif
 
 #ifdef TO_OSX // OSX/PPC
 #define OLD_COMPILER
+#define USE_SETENV 
 #endif
 
 #ifdef TO_OBSD // OpenBSD

diff --git a/src/os/posix/host-lib.c b/src/os/posix/host-lib.c
@@ -495,7 +495,7 @@ int pipe2(int pipefd[2], int flags); //to avoid "implicit-function-declaration"
 ***********************************************************************/
 {
  if (envval) {
-#ifdef setenv
+#ifdef USE_SETENV
  // we pass 1 for overwrite (make call to OS_Get_Env if you 
  // want to check if already exists)
 
@@ -530,7 +530,7 @@ int pipe2(int pipefd[2], int flags); //to avoid "implicit-function-declaration"
  return TRUE;
  }
 
-#ifdef unsetenv
+#ifdef USE_SETENV
  if (unsetenv(envname) == -1)
  return FALSE;
 #else