From 81133e613b12ca0b5fa4aa03c0a3441f1e41b1ad Mon Sep 17 00:00:00 2001 From: zecke Date: Sun, 10 Oct 2004 16:43:49 +0000 Subject: 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. --- (limited to 'noncore/apps/opie-console') 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; } -- cgit v0.9.0.2