Correct the use of OProcess.

1 files changed, 4 insertions, 10 deletions

diff --git a/noncore/settings/networksettings/wlan/wlanimp.cpp b/noncore/settings/networksettings/wlan/wlanimp.cpp
index db03266..3797e22 100644
--- a/noncore/settings/networksettings/wlan/wlanimp.cpp
+++ b/noncore/settings/networksettings/wlan/wlanimp.cpp
@@ -205,44 +205,38 @@ void WLANImp::changeAndSaveSettingFile(){
 /**
  * Check to see if the current config is valid
  * Save wireless.opts, save interfaces
  */
 void WLANImp::accept(){
   if(wepEnabled->isChecked()){
     if(keyLineEdit0->text().isEmpty() && keyLineEdit1->text().isEmpty() && keyLineEdit2->text().isEmpty() && keyLineEdit3->text().isEmpty() ){
       QMessageBox::information(this, "Error", "Please enter a key for WEP.", QMessageBox::Ok);
       return;
     }
           }
   
   if(essAny->isChecked() && essSpecificLineEdit->text().isEmpty()){
     QMessageBox::information(this, "Error", "Please enter a ESS-ID.", QMessageBox::Ok);
     return;
   }
   
   // Ok settings are good here, save
   changeAndSaveSettingFile();
   
   // Try to save the interfaces settings.
   if(!interfaceSetup->saveChanges())
     return;
 
-  OProcess insert;
-//  OProcess eject;
-
-  insert << "sh -c \"cardctl insert && cardctl eject\"";
-//  eject << "cardctl eject";
-
-//  if (!eject.start(OProcess::Block, OProcess::NoCommunication) ) {
-//    qWarning("could not start cardctl eject");
-//  }
+  OProcess insert(QString("sh"));
+  insert << "-c";
+  insert << "cardctl eject && cardctl insert";
 
   if (!insert.start(OProcess::DontCare, OProcess::NoCommunication) ) {
-    qWarning("could not start cardctl insert");
+    qWarning("could not start cardctl");
   }
 
   // Close out the dialog
   QDialog::accept();
 }
 
 // wlanimp.cpp