fix memleaks

fix possible crashes
clean up

2 files changed, 132 insertions, 160 deletions

diff --git a/noncore/net/opietooth/manager/bluebase.cpp b/noncore/net/opietooth/manager/bluebase.cpp
index 2001db1..ff6981b 100644
--- a/noncore/net/opietooth/manager/bluebase.cpp
+++ b/noncore/net/opietooth/manager/bluebase.cpp
@@ -46,3 +46,3 @@
 
-namespace OpieTooth {
+using namespace OpieTooth;
 
@@ -51,3 +51,3 @@ namespace OpieTooth {
 
-        localDevice = new Manager( "hci0" );
+        m_localDevice = new Manager( "hci0" );
 
@@ -62,7 +62,7 @@ namespace OpieTooth {
                  this,  SLOT(startServiceActionHold( QListViewItem *, const QPoint &, int) ) );
-        connect( localDevice , SIGNAL( foundServices( const QString& , Services::ValueList ) ),
+        connect( m_localDevice , SIGNAL( foundServices( const QString& , Services::ValueList ) ),
                  this, SLOT( addServicesToDevice( const QString& , Services::ValueList ) ) );
-        connect( localDevice, SIGNAL( available( const QString&, bool ) ),
+        connect( m_localDevice, SIGNAL( available( const QString&, bool ) ),
                  this, SLOT( deviceActive( const QString& , bool ) ) );
-        connect( localDevice, SIGNAL( connections( Connection::ValueList ) ),
+        connect( m_localDevice, SIGNAL( connections( Connection::ValueList ) ),
                  this, SLOT( addConnectedDevices( Connection::ValueList ) ) );
@@ -70,4 +70,4 @@ namespace OpieTooth {
         //Load all icons needed
-        offPix = Resource::loadPixmap( "editdelete" );
-        onPix = Resource::loadPixmap( "installed" );
+        m_offPix = Resource::loadPixmap( "editdelete" );
+        m_onPix = Resource::loadPixmap( "installed" );
 
@@ -90,3 +90,3 @@ namespace OpieTooth {
         BTListItem *topLV2 = new BTListItem( ListView2, "Siemens S45", "", "device" );
-        topLV2->setPixmap( 1, onPix );
+        topLV2->setPixmap( 1, m_onPix );
         (void) new BTListItem( topLV2, "Serial" ,"", "service" );
@@ -97,3 +97,3 @@ namespace OpieTooth {
         addConnectedDevices();
-        iconLoader = new BTIconLoader();
+        m_iconLoader = new BTIconLoader();
       }
@@ -108,8 +108,8 @@ namespace OpieTooth {
 
-        deviceName = cfg.readEntry( "name" , "No name" ); // name the device should identify with
-        defaultPasskey = cfg.readEntryCrypt( "passkey" , "" ); // <- hmm, look up how good the trolls did that, maybe too weak
-        useEncryption = cfg.readNumEntry( "useEncryption" , 1 );
-        enableAuthentification = cfg.readNumEntry( "enableAuthentification" , 1 );
-        enablePagescan = cfg.readNumEntry( "enablePagescan" , 1 );
-        enableInquiryscan = cfg.readNumEntry( "enableInquiryscan" , 1 );
+        m_deviceName = cfg.readEntry( "name" , "No name" ); // name the device should identify with
+        m_defaultPasskey = cfg.readEntryCrypt( "passkey" , "" ); // <- hmm, look up how good the trolls did that, maybe too weak
+        m_useEncryption = cfg.readBoolEntry( "useEncryption" , TRUE );
+        m_enableAuthentification = cfg.readBoolEntry( "enableAuthentification" , TRUE );
+        m_enablePagescan = cfg.readBoolEntry( "enablePagescan" , TRUE );
+        m_enableInquiryscan = cfg.readBoolEntry( "enableInquiryscan" , TRUE );
     }
@@ -124,8 +124,8 @@ namespace OpieTooth {
 
-        cfg.writeEntry( "name" , deviceName );
-        cfg.writeEntryCrypt( "passkey" , defaultPasskey );
-        cfg.writeEntry( "useEncryption" , useEncryption );
-        cfg.writeEntry( "enableAuthentification" , enableAuthentification );
-        cfg.writeEntry( "enablePagescan" , enablePagescan );
-        cfg.writeEntry( "enableInquiryscan" , enableInquiryscan );
+        cfg.writeEntry( "name" , m_deviceName );
+        cfg.writeEntryCrypt( "passkey" , m_defaultPasskey );
+        cfg.writeEntry( "useEncryption" , m_useEncryption );
+        cfg.writeEntry( "enableAuthentification" , m_enableAuthentification );
+        cfg.writeEntry( "enablePagescan" , m_enablePagescan );
+        cfg.writeEntry( "enableInquiryscan" , m_enableInquiryscan );
 
@@ -136,4 +136,4 @@ namespace OpieTooth {
 
-        HciConfWrapper *hciconf = new HciConfWrapper( "/tmp/hcid.conf" );
-        hciconf->setPinHelper( "/bin/QtPalmtop/bin/blue-pin" );
+        HciConfWrapper hciconf ( "/tmp/hcid.conf" );
+        hciconf.setPinHelper( "/bin/QtPalmtop/bin/blue-pin" );
 
@@ -142,29 +142,7 @@ namespace OpieTooth {
 
-        hciconf->setName( deviceName );
-
-        if ( useEncryption == 1) {
-            hciconf->setEncrypt( true );
-        } else {
-            hciconf->setEncrypt( false );
-        }
-
-
-        if ( enableAuthentification == 1) {
-            hciconf->setAuth( true );
-        } else {
-            hciconf->setAuth( false );
-        }
-
-        if ( enablePagescan  == 1) {
-            hciconf->setPscan( true );
-        } else {
-            hciconf->setPscan( false );
-        }
-
-        if ( enableInquiryscan  == 1) {
-            hciconf->setIscan( true );
-        } else {
-            hciconf->setIscan( false );
-        }
-	delete hciconf;
+        hciconf.setName( m_deviceName );
+        hciconf.setEncrypt( m_useEncryption );
+	hciconf.setAuth( m_enableAuthentification );
+        hciconf.setPscan( m_enablePagescan );
+        hciconf.setIscan( m_enableInquiryscan );
     }
@@ -178,3 +156,3 @@ namespace OpieTooth {
 
-        QList<RemoteDevice> *loadedDevices = new QList<RemoteDevice>;
+        QValueList<RemoteDevice> loadedDevices;
 
@@ -196,6 +174,6 @@ namespace OpieTooth {
             qDebug("NAME: " + name);
-            RemoteDevice currentDevice = RemoteDevice( mac , name  );
-            loadedDevices->append( &currentDevice );
+            RemoteDevice currentDevice( mac , name  );
+            loadedDevices.append( currentDevice );
         }
-        addSearchedDevices( *loadedDevices );
+        addSearchedDevices( loadedDevices );
     }
@@ -223,9 +201,9 @@ namespace OpieTooth {
     void BlueBase::initGui() {
-        StatusLabel->setText( getStatus() ); // maybe move it to getStatus()
-        cryptCheckBox->setChecked( useEncryption );
-        authCheckBox->setChecked( enableAuthentification );
-        pagescanCheckBox->setChecked( enablePagescan );
-        inquiryscanCheckBox->setChecked( enableInquiryscan );
-        deviceNameLine->setText( deviceName );
-        passkeyLine->setText( defaultPasskey );
+        StatusLabel->setText( status() ); // maybe move it to getStatus()
+        cryptCheckBox->setChecked( m_useEncryption );
+        authCheckBox->setChecked( m_enableAuthentification );
+        pagescanCheckBox->setChecked( m_enablePagescan );
+        inquiryscanCheckBox->setChecked( m_enableInquiryscan );
+        deviceNameLine->setText( m_deviceName );
+        passkeyLine->setText( m_defaultPasskey );
         // set info tab
@@ -239,3 +217,3 @@ namespace OpieTooth {
      */
-    QString BlueBase::getStatus(){
+    QString BlueBase::status()const{
         QString infoString = tr( "<b>Device name : </b> Ipaq" );
@@ -252,8 +230,8 @@ namespace OpieTooth {
     void BlueBase::applyConfigChanges() {
-        deviceName =  deviceNameLine->text();
-        defaultPasskey = passkeyLine->text();
-        useEncryption = cryptCheckBox->isChecked();
-        enableAuthentification = authCheckBox->isChecked();
-        enablePagescan = pagescanCheckBox->isChecked();
-        enableInquiryscan = inquiryscanCheckBox->isChecked();
+        m_deviceName =  deviceNameLine->text();
+        m_defaultPasskey = passkeyLine->text();
+        m_useEncryption = cryptCheckBox->isChecked();
+        m_enableAuthentification = authCheckBox->isChecked();
+        m_enablePagescan = pagescanCheckBox->isChecked();
+        m_enableInquiryscan = inquiryscanCheckBox->isChecked();
 
@@ -261,5 +239,3 @@ namespace OpieTooth {
 
-        QMessageBox*  box = new QMessageBox( this, "Test" );
-        box->setText( tr( "Changes applied" ) );
-        box->show();
+        QMessageBox::information( this, tr("Test") , tr("Changes were applied.") );
    }
@@ -270,10 +246,8 @@ namespace OpieTooth {
      */
-    void BlueBase::addSearchedDevices( QList<RemoteDevice> &newDevices ) {
+    void BlueBase::addSearchedDevices( const QValueList<RemoteDevice> &newDevices ) {
         BTListItem * deviceItem;
-        QListIterator<RemoteDevice> it( newDevices );
+        QValueList<RemoteDevice>::ConstIterator it;
 
-        for( ; it.current() ; ++it ) {
-
-            RemoteDevice *dev = it.current();
-            deviceItem = new BTListItem( ListView2 , dev->name(), dev->mac(), "device" );
+        for( it = newDevices.begin(); it != newDevices.end() ; ++it ) {
+            deviceItem = new BTListItem( ListView2 , (*it).name(), (*it).mac(), "device" );
             deviceItem->setExpandable ( true );
@@ -281,3 +255,3 @@ namespace OpieTooth {
             // look if device is avail. atm, async
-            deviceActive( dev );
+            deviceActive( (*it) );
 
@@ -292,4 +266,4 @@ namespace OpieTooth {
      */
-    void BlueBase::startServiceActionClicked( QListViewItem *item ) {
-    }
+void BlueBase::startServiceActionClicked( QListViewItem */*item*/ ) {
+}
 
@@ -298,53 +272,53 @@ namespace OpieTooth {
      */
-    void BlueBase::startServiceActionHold( QListViewItem * item, const QPoint & point, int column ) {
-
-          QPopupMenu *menu = new QPopupMenu();
-          int ret=0;
-
-          if ( ((BTListItem*)item)->type() == "device") {
-
-              QPopupMenu *groups = new QPopupMenu();
-
-              menu->insertItem( tr("rescan sevices:"),  0);
-              menu->insertItem( tr("to group"), groups ,  1);
-              menu->insertItem( tr("bound device"), 2);
-              menu->insertItem( tr("delete"),  3);
-
-              ret = menu->exec( point  , 0);
-
-              switch(ret) {
-              case 0:
-                  break;
-              case 1:
-                  break;
-              case 2:
-                  // make connection
-                  break;
-              case 3:
-                  // delete childs too
-                  delete item;
-                  break;
-              }
-              delete groups;
-
-          } else if ( ((BTListItem*)item)->type() == "service") {
-              menu->insertItem( tr("Test1:"),  0);
-              menu->insertItem( tr("connect"), 1);
-              menu->insertItem( tr("delete"),  2);
-
-              ret = menu->exec( point  , 0);
-
-              switch(ret) {
-              case 0:
-                  break;
-              case 1:
-                  break;
-              case 2:
-                  // delete childs too
-                  delete item;
-                  break;
-              }
-          }
-          delete menu;
+void BlueBase::startServiceActionHold( QListViewItem * item, const QPoint & point, int /*column*/ ) {
+
+    QPopupMenu *menu = new QPopupMenu();
+    int ret=0;
+
+    if ( ((BTListItem*)item)->type() == "device") {
+
+        QPopupMenu *groups = new QPopupMenu();
+
+        menu->insertItem( tr("rescan sevices:"),  0);
+        menu->insertItem( tr("to group"), groups ,  1);
+        menu->insertItem( tr("bound device"), 2);
+        menu->insertItem( tr("delete"),  3);
+
+        ret = menu->exec( point  , 0);
+
+        switch(ret) {
+        case 0:
+            break;
+        case 1:
+            break;
+        case 2:
+            // make connection
+            break;
+        case 3:
+            // delete childs too
+            delete item;
+            break;
+        }
+        delete groups;
+
+    } else if ( ((BTListItem*)item)->type() == "service") {
+        menu->insertItem( tr("Test1:"),  0);
+        menu->insertItem( tr("connect"), 1);
+        menu->insertItem( tr("delete"),  2);
+
+        ret = menu->exec( point  , 0);
+
+        switch(ret) {
+        case 0:
+            break;
+        case 1:
+            break;
+        case 2:
+            // delete childs too
+            delete item;
+            break;
+        }
     }
+    delete menu;
+}
 
@@ -357,7 +331,6 @@ namespace OpieTooth {
         // row of mac adress text(3)
-        RemoteDevice *device = new RemoteDevice( item->mac(),  item->name() );
-        deviceList.insert( item->mac() ,  item );
+        RemoteDevice device( item->mac(),  item->name() );
+        m_deviceList.insert( item->mac() ,  item );
         // and some time later I get a signal foundServices( const QString& device, Services::ValueList ); back
-        localDevice->searchServices( *device );
-        delete device;
+        m_localDevice->searchServices( device );
     }
@@ -377,3 +350,3 @@ namespace OpieTooth {
         // get the right devices which requested the search
-        for( it = deviceList.begin(); it != deviceList.end(); ++it ) {
+        for( it = m_deviceList.begin(); it != m_deviceList.end(); ++it ) {
             if ( it.key() == device ) {
@@ -390,3 +363,3 @@ namespace OpieTooth {
                 serviceItem = new BTListItem( deviceItem  ,  (*it2).serviceName() , "" , "service"  );
-                serviceItem->setPixmap( 0, iconLoader->serviceIcon( (*it2).classIdList() ) );
+                serviceItem->setPixmap( 0, m_iconLoader->serviceIcon( (*it2).classIdList() ) );
             }
@@ -403,3 +376,3 @@ namespace OpieTooth {
     void BlueBase::addConnectedDevices() {
-        localDevice->searchConnections();
+        m_localDevice->searchConnections();
     }
@@ -427,5 +400,5 @@ namespace OpieTooth {
      */
-    void BlueBase::deviceActive( RemoteDevice *device ) {
+    void BlueBase::deviceActive( const RemoteDevice &device ) {
         // search by mac, async, gets a signal back
-        localDevice->isAvailable( device->mac() );
+        m_localDevice->isAvailable( device.mac() );
     }
@@ -444,3 +417,3 @@ namespace OpieTooth {
         // get the right devices which requested the search
-        for( it = deviceList.begin(); it != deviceList.end(); ++it ) {
+        for( it = m_deviceList.begin(); it != m_deviceList.end(); ++it ) {
             if ( it.key() == device ) {
@@ -451,5 +424,5 @@ namespace OpieTooth {
         if ( connected ) {
-            deviceItem->setPixmap( 1, onPix );
+            deviceItem->setPixmap( 1, m_onPix );
         } else {
-            deviceItem->setPixmap( 1, offPix );
+            deviceItem->setPixmap( 1, m_offPix );
         }
@@ -473,3 +446,3 @@ namespace OpieTooth {
     void BlueBase::setInfo() {
-        StatusLabel->setText( getStatus() );
+        StatusLabel->setText( status() );
     }
@@ -481,5 +454,4 @@ namespace OpieTooth {
         writeSavedDevices();
-        delete iconLoader;
+        delete m_iconLoader;
     }
-}
 
diff --git a/noncore/net/opietooth/manager/bluebase.h b/noncore/net/opietooth/manager/bluebase.h
index ce0483f..73fac97 100644
--- a/noncore/net/opietooth/manager/bluebase.h
+++ b/noncore/net/opietooth/manager/bluebase.h
@@ -40,3 +40,3 @@ namespace OpieTooth {
 
-            private slots:
+    private slots:
         void startScan();
@@ -49,24 +49,24 @@ namespace OpieTooth {
         void writeToHciConfig();
-	QString getStatus();
+	QString status()const;
 	void initGui();
         void setInfo();
-        Manager *localDevice;
-        QMap<QString,BTListItem*> deviceList;
+        Manager *m_localDevice;
+        QMap<QString,BTListItem*> m_deviceList;
 
-        void deviceActive( RemoteDevice *device );
+        void deviceActive( const RemoteDevice &device );
 
-	QString deviceName;
-	QString defaultPasskey;
-	int useEncryption;
-	int enableAuthentification;
-	int enablePagescan;
-	int enableInquiryscan;
+	QString m_deviceName;
+	QString m_defaultPasskey;
+	bool m_useEncryption;
+	bool m_enableAuthentification;
+	bool m_enablePagescan;
+	bool m_enableInquiryscan;
 
-        QPixmap offPix;
-        QPixmap onPix;
+        QPixmap m_offPix;
+        QPixmap m_onPix;
 
-        BTIconLoader *iconLoader;
+        BTIconLoader *m_iconLoader;
 
 	private slots:
-        void addSearchedDevices( QList<RemoteDevice> &newDevices );
+        void addSearchedDevices( const QValueList<RemoteDevice> &newDevices );
         void addServicesToDevice( BTListItem *item );