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
@@ -66,7 +66,5 @@ void FixIt::fixIt() {
     
     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";
     
     
@@ -83,10 +81,8 @@ 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; 
      }