From 8c1908035c919da58d29fab87b42126754f84405 Mon Sep 17 00:00:00 2001

From: petervo <petervo@redhat.com>

Date: Wed, 10 Feb 2016 02:29:00 -0800

Subject: [PATCH] session: Make sure we set path in cockpit-session

Reviewed-by: Marius Vollmer <marius.vollmer@redhat.com>

---

 src/ws/session.c | 7 ++++++-

 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/src/ws/session.c b/src/ws/session.c

index e877f15..618ca2e 100644

--- a/src/ws/session.c

+++ b/src/ws/session.c

@@ -56,6 +56,7 @@

 #define MAX_BUFFER 64 * 1024

 #define AUTH_FD 3

 #define EX 127

+#define DEFAULT_PATH "/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin"

 static struct passwd *pwd;

 const char *rhost;

@@ -898,6 +899,7 @@ static const char *env_names[] = {

   "G_DEBUG",

   "G_MESSAGES_DEBUG",

   "G_SLICE",

+  "PATH",

   NULL

 };

@@ -910,6 +912,9 @@ save_environment (void)

   const char *value;

   int i, j;

+  /* Force save our default path */

+  setenv ("PATH", DEFAULT_PATH, 1);

+

   for (i = 0, j = 0; env_names[i] != NULL; i++)

     {

       value = getenv (env_names[i]);

@@ -957,7 +962,7 @@ main (int argc,

         }

       /* set a minimal environment */

-      setenv ("PATH", "/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin", 1);

+      setenv ("PATH", DEFAULT_PATH, 1);

       if (setgid (0) != 0 || setuid (0) != 0)

         err (1, "couldn't switch permissions correctly");

-- 

1.8.3.1