Use setenv instead of putenv

In theory putenv could be used to handle freeing of strings yourself, but this
was not done in PulseAudio.  That leaves no advantages in using putenv.  With
setenv you're at the mercy of the implementation whether the strings leak, but
at least that is better then a certain leak, as it was before.

diff --git a/src/pulsecore/core-util.c b/src/pulsecore/core-util.c
index 4e7d0d7..ec0b338 100644 (file)
--- a/src/pulsecore/core-util.c
+++ b/src/pulsecore/core-util.c
@@ -2501,7 +2501,7 @@ void pa_set_env(const char *key, const char *value) {
 
     /* This is not thread-safe */
 
-    putenv(pa_sprintf_malloc("%s=%s", key, value));
+    setenv(key, value, 1);
 }
 
 void pa_set_env_and_record(const char *key, const char *value) {