2 files changed, 7 insertions, 12 deletions

diff --git a/noncore/unsupported/gsmtool/gsmtool.cpp b/noncore/unsupported/gsmtool/gsmtool.cpp
index 14ef368..38c8c59 100644
--- a/noncore/unsupported/gsmtool/gsmtool.cpp
+++ b/noncore/unsupported/gsmtool/gsmtool.cpp
@@ -19,13 +19,12 @@ using namespace gsmlib;
  */
 GSMTool::GSMTool( QWidget* parent,  const char* name, WFlags fl )
     : GSMToolBase( parent, name, fl )
 {
         devicelocked = 0;
         me = NULL;
-        port = NULL;
         setConnected(FALSE);
         /* FIXME: Persistent settings for device/baudrate */
         connect(ConnectButton, SIGNAL(clicked()), this, SLOT(doConnectButton()));
         connect(ScanButton, SIGNAL(clicked()), this, SLOT(doScanButton()));
         connect(TabWidget2, SIGNAL(currentChanged(QWidget *)), this, SLOT(doTabChanged()));
         timerid = -1; // Is this not possible normally? 
@@ -135,52 +134,51 @@ void GSMTool::doScanButton()
 }
 /*
  *  A simple slot... not very interesting.
  */
 void GSMTool::doConnectButton()
 {
+        gsmlib::Port *port;
+
         speed_t rate;
         devicename = strdup(DeviceName->currentText().local8Bit().data());
         rate = baudrates[BaudRate->currentItem()];
 
         qDebug("Connect Button Pressed");
         MfrText->setText("Opening...");
         ModelText->setText("");
         RevisionText->setText("");
         SerialText->setText("");
 
         setConnected(FALSE);
         if (me) {
-                        //      delete me;
                 me = NULL;
         }
-        if (port) {
-                        //      delete port;
-                port = NULL;
-        }
 
         if (lockDevice()) {
                 qDebug("lockDevice() failed\n");
+                MfrText->setText("Lock port failed");
         };
         
         qDebug("Device name is %s\n", devicename);
 
         try {
                 port = new UnixSerialPort(devicename, rate, DEFAULT_INIT_STRING, 0);
         } catch (GsmException) {
                 qDebug("port failed");
+                MfrText->setText("Open port failed");
                 return;
         }
         MfrText->setText("Initialising...");
         qDebug("got port");
         try {
                 me = new MeTa(port);
         } catch (GsmException) {
                 qDebug("meta failed");
-                delete port;
-                port = NULL;
+                MfrText->setText("Initialise GSM unit failed");
+                me = NULL;
                 unlockDevice();
                 return;
         }
 
         qDebug("Opened");
 
@@ -189,16 +187,14 @@ void GSMTool::doConnectButton()
         MfrText->setText("Querying...");
 
         try {
                 ifo = me->getMEInfo();
         } catch (GsmException) {
                 qDebug("getMEInfo failed");
-                delete me;
+                MfrText->setText("Query GSM unit failed");
                 me = NULL;
-                delete port;
-                port = NULL;
                 unlockDevice();
                 return;
         }
 
         MfrText->setText(ifo._manufacturer.c_str());
         ModelText->setText(ifo._model.c_str());
diff --git a/noncore/unsupported/gsmtool/gsmtool.h b/noncore/unsupported/gsmtool/gsmtool.h
index cb19f54..1625cb1 100644
--- a/noncore/unsupported/gsmtool/gsmtool.h
+++ b/noncore/unsupported/gsmtool/gsmtool.h
@@ -24,13 +24,12 @@ private slots:
 private:
  static const speed_t baudrates[];
  int devicelocked;
  int timerid;
 
  gsmlib::MeTa *me;
- gsmlib::Port *port;
 
  char *devicename; 
  speed_t baudrate;
 
  int lockDevice( );
  void unlockDevice( );