From ef533f251490c53a31b51ca0e18c6554446fd939 Mon Sep 17 00:00:00 2001
From: pQu4k3r <paulo23rocha@gmail.com>
Date: Tue, 11 Aug 2026 14:28:14 +0000
Subject: [PATCH] Run console commands via a temp script (fixes scripts not
 executing)

eConsoleAppContainer.execute() is not consistent across images: some
run the given string through a shell, some split it on whitespace. The
previous approach wrapped remote commands in /bin/sh -c "..." but
exempted anything starting with "sh " - so local sh/ scripts reached
execute() as 'sh "..." > log 2>&1' with the redirect intact, and on a
word-splitting image the redirect tokens became literal arguments and
the script silently failed. Remote pipelines were similarly fragile.

Instead of guessing how execute() parses the string, write the whole
command to a real temp script (/tmp/.lsconsole_<run>.sh) and run
'/bin/sh <script>'. A script file is always interpreted by /bin/sh
exactly as written, so pipes, redirects, && chains and embedded quotes
(the ?inline=false URLs) all work identically on every image. The
multi-command path in runFinished now goes through the same helper
(it previously executed raw, bypassing any wrapping).

Verified by executing all four command shapes (piped remote, embedded
quotes, opkg && chain, local sh script with redirect) through the real
shell.
---
 .../Extensions/LinuxsatPanel/lsConsole.py     | 27 ++++++++++++++++---
 1 file changed, 23 insertions(+), 4 deletions(-)

diff --git a/usr/lib/enigma2/python/Plugins/Extensions/LinuxsatPanel/lsConsole.py b/usr/lib/enigma2/python/Plugins/Extensions/LinuxsatPanel/lsConsole.py
index 87dca218..121d87d7 100644
--- a/usr/lib/enigma2/python/Plugins/Extensions/LinuxsatPanel/lsConsole.py
+++ b/usr/lib/enigma2/python/Plugins/Extensions/LinuxsatPanel/lsConsole.py
@@ -110,14 +110,33 @@ def __init__(
     def updateTitle(self):
         self.setTitle(self.newtitle)
 
+    def execScript(self, cmd):
+        """Run any command line reliably.
+
+        eConsoleAppContainer.execute() behaves differently across images
+        (some run the string through a shell, some split it on spaces),
+        so pipes, redirects and quotes are unreliable when passed
+        directly. Writing the command to a real temp script and running
+        that removes every one of those differences - a script file is
+        always interpreted by /bin/sh exactly as written.
+        """
+        script = '/tmp/.lsconsole_%d.sh' % self.run
+        try:
+            with codecs.open(script, 'w', encoding='utf-8') as f:
+                f.write('#!/bin/sh\n')
+                f.write(cmd + '\n')
+            os.chmod(script, 0o755)
+        except (IOError, OSError) as e:
+            print('[Console] cannot write run script:', e)
+            return self.container.execute(cmd)  # last-resort fallback
+        return self.container.execute('/bin/sh ' + script)
+
     def startRun(self):
         if self.showStartStopText:
             self['text'].setText(_('Execution progress\n\n'))
         cmd = self.cmdlist[self.run]
-        if not cmd.startswith('sh ') and not cmd.startswith('/bin/sh '):
-            cmd = '/bin/sh -c "{}"'.format(cmd.replace('"', '\\"'))
         print('[Console] executing:', cmd)
-        if self.container.execute(cmd):
+        if self.execScript(cmd):
             self['text'].setText(cmd)
             self.runFinished(-1)
 
@@ -129,7 +148,7 @@ def runFinished(self, retval):
         self.run += 1
 
         if self.run != len(self.cmdlist):
-            if self.container.execute(self.cmdlist[self.run]):
+            if self.execScript(self.cmdlist[self.run]):
                 self.runFinished(-1)
             return  # Exit early