Issue8: added deleted PublishMessage un PublishMessageAction to avoid…

… memory leak
alexCajas · Nov 6, 2023 · 4178aab · 4178aab
1 parent 70c596b
commit 4178aab
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/src/Actions/PublishAction.cpp b/src/Actions/PublishAction.cpp
@@ -4,6 +4,11 @@ PublishAction::PublishAction(MqttClient* mqttClient, ReaderMqttPacket packetRead
     publishMqttMessage = new PublishMqttMessage(packetReaded);
 }
 
+PublishAction::~PublishAction(){
+    delete publishMqttMessage;
+}
+
+
 void PublishAction::doAction(){
     mqttClient->notifyPublishRecived(publishMqttMessage);
 }
diff --git a/src/MqttBroker/MqttBroker.h b/src/MqttBroker/MqttBroker.h
@@ -418,7 +418,7 @@ class PublishAction: public Action{
          * PublishMqttMessage object.
          */
         PublishAction(MqttClient* mqttClient, ReaderMqttPacket packetReaded);
-
+        ~PublishAction();
         /**
          * @brief Notify to Broker class a publish mqtt request
          * recevided from this client.