Respect custom LESS/MORE/LV variables

Use setenv() to avoid overriding user defined variables.

LP: #2103921

1 files changed, 11 insertions, 1 deletions

diff --git a/apt-private/private-output.cc b/apt-private/private-output.cc
index 4480d7382..a1105be61 100644
--- a/apt-private/private-output.cc
+++ b/apt-private/private-output.cc
@@ -160,7 +160,17 @@ bool InitOutputPager()
 	 signal(sig, SIG_DFL);
 
       for (auto &v: pagerEnv)
-	 putenv(v.data());
+      {
+	 // WARNING: VectorizeString() is not thread-safe, do not call after creating a thread.
+	 auto keyAndValue = VectorizeString(v, '=');
+	 if (keyAndValue.size() != 2)
+	 {
+	    _error->Fatal("Invalid environment string: %s", v.c_str());
+	    goto err;
+	 }
+	 // NOTE: Must not override user values, so setenv(..., override=0) is used rather than putenv()
+	 setenv(keyAndValue[0].c_str(), keyAndValue[1].c_str(), 0);
+      }
 
       {
 	 // If our pager name contains a space we need to invoke it in a shell. Boooo!