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 15cc0a7f7b75724f07f3ca28ba283dc8efc8d454..af75296f3f0ba9cd5e96d45136abd345c03613e3 100755 (executable)
--- a/src/sessiond/tlm-session.c
+++ b/src/sessiond/tlm-session.c
@@ -423,7 +423,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) {
@@ -431,7 +431,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);
     }