commit 69abb97e547ce01df72f590cd956f4d2d5280628
author Ilya Moiseenko <iliamo@cs.ucla.edu> Mon Nov 04 16:02:20 2013 -0800
committer Alexander Afanasyev <alexander.afanasyev@ucla.edu> Fri Nov 08 22:10:10 2013 -0800
tree 3814fffbf057e20bf3a7d74071d02791e2e4d245
parent a822b57fea321caa78a092445071eb1af55789ef

Fixed memory leaks and UI improvements

Change-Id: I830d838b287915e37fe018787a85c9a2dfb7a3a7

linux/network-manager.cpp

diff --git a/linux/network-manager.cpp b/linux/network-manager.cpp
index f101f51..e416186 100644
--- a/linux/network-manager.cpp
+++ b/linux/network-manager.cpp
@@ -6,17 +6,15 @@
  */
 
 #include "network-manager.h"
-#include <QProcess>
 
 NetworkManager::NetworkManager()
 {
     if (!QDBusConnection::systemBus().isConnected())
     {
-        qDebug() << "Cannot connect to the D-Bus session bus.\n";
         return;
     }
 
-    autoconfigProcess = new QProcess();
+    autoconfigProcess = new QProcess(this);
     connect(autoconfigProcess,SIGNAL(finished(int)),this,SLOT(autoconfigFinished()));
     connect(autoconfigProcess, SIGNAL(error(QProcess::ProcessError)), this, SLOT(autoconfigFinished()));
 
@@ -28,8 +26,6 @@
 
 void NetworkManager::stateChanged(uint state)
 {
-    qDebug() << "State = " << state;
-
     if(state == NM_STATE_CONNECTED_GLOBAL)
         autoconfigDaemon();
 }
@@ -39,14 +35,14 @@
     if(IsAutoconfigRunning())
         return;
 
-     qDebug() << "started running";
-     isAutoconfigRunning = true;
-     autoconfigProcess->start(NDND_AUTOCONFIG_COMMAND);
+    isAutoconfigRunning = true;
+
+    if(autoconfigProcess != NULL)
+        autoconfigProcess->start(NDND_AUTOCONFIG_COMMAND);
 }
 
 void NetworkManager::autoconfigFinished()
 {
-    qDebug() << "stoped running";
     isAutoconfigRunning = false;
 }