From 549a23ffb73536032306fd08ea777c772300d5f6 Mon Sep 17 00:00:00 2001
From: hsaturn <hsaturn@gmail.com>
Date: Wed, 31 Mar 2021 10:36:51 +0200
Subject: [PATCH] Fix delete was not really deleting in tinytest

---
 examples/tinymqtt-test/tinymqtt-test.ino | 4 ++--
 src/TinyMqtt.cpp                         | 1 +
 2 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/examples/tinymqtt-test/tinymqtt-test.ino b/examples/tinymqtt-test/tinymqtt-test.ino
index 866593a..9d112e9 100644
--- a/examples/tinymqtt-test/tinymqtt-test.ino
+++ b/examples/tinymqtt-test/tinymqtt-test.ino
@@ -399,11 +399,11 @@ void loop()
 					}
 					if (client)
 					{
-						clients.erase(s);
 						for (auto it: clients)
 						{
 							if (it.second != client) continue;
 							Serial << "deleted" << endl;
+							delete (it.second);
 							clients.erase(it.first);
 							break;
 						}
@@ -413,9 +413,9 @@ void loop()
 					{
 						for(auto it: brokers)
 						{
-							Serial << (int32_t)it.second << '/' << (int32_t)broker << endl;
 							if (broker != it.second) continue;
 							Serial << "deleted" << endl;
+							delete (it.second);
 							brokers.erase(it.first);
 							break;
 						}
diff --git a/src/TinyMqtt.cpp b/src/TinyMqtt.cpp
index 3a95f39..9e3f3ce 100644
--- a/src/TinyMqtt.cpp
+++ b/src/TinyMqtt.cpp
@@ -19,6 +19,7 @@ MqttBroker::~MqttBroker()
 	{
 		delete clients[0];
 	}
+	server.close();
 }
 
 // private constructor used by broker only