Do not 'spawn' a shell this way. Use OProcess::setUseShell if you really

need to start the command via a shell.

These scripts should be marked as executable and have !#/bin/sh in the
header, this should be enough for the kernel to launch them via sh.

1 files changed, 3 insertions, 7 deletions

diff --git a/noncore/apps/opie-console/fixit.cpp b/noncore/apps/opie-console/fixit.cpp
index 5ac7673..57dba4c 100644
--- a/noncore/apps/opie-console/fixit.cpp
+++ b/noncore/apps/opie-console/fixit.cpp
@@ -65,9 +65,7 @@ void FixIt::fixIt() {
     #else
     
     OProcess m_kill;
-    m_kill << "sh";
-    m_kill << "-c";
-    m_kill << QString(QPEApplication::qpeDir() + "/share/opie-console/sl6000_embedix_kill_0_1.sh");
+    m_kill << QPEApplication::qpeDir() + "/share/opie-console/sl6000_embedix_kill_0_1.sh";
     
     
     if ( !m_kill.start(OProcess::DontCare,OProcess::NoCommunication) ) {
@@ -82,12 +80,10 @@ void FixIt::fixIt() {
 void FixIt::breakIt() {
 	#ifdef EAST
 	 OProcess m_restart;
-         m_restart << "sh";
-         m_restart << "-c";
-         m_restart << QString(QPEApplication::qpeDir() + "/share/opie-console/sl6000_embedix_restart_0_1.sh");
+         m_restart << QPEApplication::qpeDir() + "/share/opie-console/sl6000_embedix_restart_0_1.sh";
          
     
-     if ( !m_restart.start() ) {
+     if ( !m_restart.start(OProcess::DontCare,OProcess::NoCommunication) ) {
             owarn << "could not execute restart script" << oendl; 
      }