MecAppBase: Eliminate undisposed objects

The ~MecAppBase() destructor in MecAppBase.cc:28 was calling
sockets_.deleteSockets() which deletes the TcpSocket objects
but does not clean up the HttpMessageStatus structs attached as
userData to each socket.

This mirrors the cleanup logic already present in removeSocket(),
which was never called during destruction.

Author: avarga

src/simu5g/apps/mec/MecApps/MecAppBase.cc

@@ -30,6 +30,20 @@ MecAppBase::~MecAppBase()
     std::cout << "MecAppBase::~MecAppBase()" << std::endl;
     cancelAndDelete(sendTimer);
 
+    for (auto &entry : sockets_.getMap()) {
+        auto *sock = dynamic_cast<inet::TcpSocket *>(entry.second);
+        if (sock == nullptr)
+            continue;
+        auto *msgStatus = static_cast<HttpMessageStatus *>(sock->getUserData());
+        if (msgStatus == nullptr)
+            continue;
+        while (!msgStatus->httpMessageQueue.isEmpty())
+            delete msgStatus->httpMessageQueue.pop();
+        delete msgStatus->currentMessage;
+        cancelAndDelete(msgStatus->processMsgTimer);
+        delete msgStatus;
+        sock->setUserData(nullptr);
+    }
     sockets_.deleteSockets();
 
     cancelAndDelete(processMessage_);