Merge pull request #32 from thunder2/pr/2437

Pr/2437

libretroshare/src/pqi/rstcpsocket.cc

@@ -84,7 +84,7 @@ RsThreadedTcpSocket::RsThreadedTcpSocket() : RsTcpSocket()
 }
 void RsThreadedTcpSocket::run()
 {
-    while(connectionState() == CONNECTED)
+    while(!shouldStop() && connectionState() == CONNECTED)
     {
         tick();
         std::this_thread::sleep_for(std::chrono::milliseconds(200));

libretroshare/src/tor/TorControl.cpp

@@ -46,8 +46,6 @@
 #include "StrUtil.h"
 #include "PendingOperation.h"
 
-Tor::TorControl *torControl = 0;
-
 class nullstream: public std::ostream {};
 
 static std::ostream& torctrldebug()
@@ -70,6 +68,11 @@ TorControl::TorControl()
     mSocket = new TorControlSocket(this);
 }
 
+TorControl::~TorControl()
+{
+    delete(mSocket);
+}
+
 static RsTorConnectivityStatus torConnectivityStatus(Tor::TorControl::Status t)
 {
     switch(t)

libretroshare/src/tor/TorControl.h

@@ -68,6 +68,7 @@ public:
     };
 
     explicit TorControl();
+    virtual ~TorControl();
 
     /* Information */
     Status status() const;
@@ -148,5 +149,3 @@ private:
 };
 
 }
-
-extern Tor::TorControl *torControl;

libretroshare/src/tor/TorManager.cpp

@@ -71,6 +71,7 @@ public:
 	HiddenService *hiddenService ;
 
     explicit TorManagerPrivate(TorManager *parent = 0);
+    virtual ~TorManagerPrivate();
 
     std::string torExecutablePath() const;
     bool createDataDir(const std::string &path);
@@ -94,6 +95,11 @@ TorManager::TorManager()
 {
 }
 
+TorManager::~TorManager()
+{
+    delete(d);
+}
+
 TorManagerPrivate::TorManagerPrivate(TorManager *parent)
     : q(parent)
     , process(0)
@@ -104,6 +110,11 @@ TorManagerPrivate::TorManagerPrivate(TorManager *parent)
     control->set_statusChanged_callback([this](int new_status,int /*old_status*/) { controlStatusChanged(new_status); });
 }
 
+TorManagerPrivate::~TorManagerPrivate()
+{
+    delete(control);
+}
+
 TorManager *TorManager::instance()
 {
     static TorManager *p = 0;

libretroshare/src/tor/TorManager.h

@@ -51,6 +51,7 @@ class TorManager : public HiddenServiceClient, public RsThread, public RsTor
 {
 public:
     static TorManager *instance();
+    virtual ~TorManager();
 
     TorProcess *process();
     TorControl *control();

retroshare-gui/src/TorControl/TorControlWindow.cpp

@@ -51,6 +51,11 @@ TorControlDialog::TorControlDialog(QWidget *)
         adjustSize();
 }
 
+TorControlDialog::~TorControlDialog()
+{
+    rsEvents->unregisterEventsHandler(mEventHandlerId);
+}
+
 void TorControlDialog::handleEvent_main_thread(std::shared_ptr<const RsEvent> event)
 {
     if(event->mType != RsEventType::TOR_MANAGER) return;

retroshare-gui/src/TorControl/TorControlWindow.h

@@ -15,6 +15,7 @@ class TorControlDialog: public QWidget, public Ui::TorControlDialog
 
 public:
     TorControlDialog(QWidget *parent =NULL);
+    virtual ~TorControlDialog();
 
 	enum TorStatus {
 		TOR_STATUS_UNKNOWN = 0x00,