Fix a bug that incorrectly uses putenv

Refer: https://stackoverflow.com/questions/5873029/questions-about-putenv-and-setenv

Change-Id: I26f1e7e6ce14f78f827a8440b97baf930b6d3102
(cherry picked from commit 43c2e3bbc25b77ab7f9d56956fc855a4f4b13788)

diff --git a/src/sessiond/tlm-session.c b/src/sessiond/tlm-session.c
index 7c8c08c044953a7313d2b663696d45207ff719d3..7782aa11dbd1c36e1c351fd1b58ffb70959c0bb1 100755 (executable)
--- a/src/sessiond/tlm-session.c
+++ b/src/sessiond/tlm-session.c
@@ -428,7 +428,7 @@ reset_exit:
 static gboolean
 _set_environment (TlmSessionPrivate *priv)
 {
-       gchar **envlist = tlm_auth_session_get_envlist(priv->auth_session);
+    gchar **envlist = tlm_auth_session_get_envlist(priv->auth_session);
     gchar *home_dir = NULL, *shell = NULL;
 
     if (envlist) {
@@ -436,7 +436,7 @@ _set_environment (TlmSessionPrivate *priv)
         for (env = envlist; *env != NULL; ++env) {
             DBG ("ENV : %s", *env);
             putenv(*env);
-            g_free (*env);
+            // Don't free *env because putenv uses pointer internally without copying the string.
         }
         g_free (envlist);
     }