unique_ptr for sent and incomplete messages

SSUData.cpp

@@ -23,10 +23,6 @@ namespace transport
 
 	SSUData::~SSUData ()
 	{
-		for (auto it: m_IncomleteMessages)
-			delete it.second;
-		for (auto it: m_SentMessages)
-			delete it.second;
 	}
 
 	void SSUData::Start ()
@@ -78,7 +74,6 @@ namespace transport
 		auto it = m_SentMessages.find (msgID);
 		if (it != m_SentMessages.end ())
 		{
-			delete it->second;
 			m_SentMessages.erase (it);	
 			if (m_SentMessages.empty ())
 				m_ResendTimer.cancel ();
@@ -161,22 +156,19 @@ namespace transport
 
 			//  find message with msgID
 			I2NPMessage * msg = nullptr;
-			IncompleteMessage * incompleteMessage = nullptr;
+			auto it = m_IncompleteMessages.find (msgID);
-			auto it = m_IncomleteMessages.find (msgID);
+			if (it != m_IncompleteMessages.end ()) 
-			if (it != m_IncomleteMessages.end ()) 
-			{	
 				// message exists
-				incompleteMessage = it->second;
+				msg = it->second->msg;
-				msg = incompleteMessage->msg;
-			}	
 			else
 			{
 				// create new message
 				msg = NewI2NPMessage ();
 				msg->len -= I2NP_SHORT_HEADER_SIZE;
-				incompleteMessage = new IncompleteMessage (msg);
+				it = m_IncompleteMessages.insert (std::make_pair (msgID, 
-				m_IncomleteMessages[msgID] = incompleteMessage;
+					std::unique_ptr<IncompleteMessage>(new IncompleteMessage (msg)))).first;
 			}
+			std::unique_ptr<IncompleteMessage>& incompleteMessage = it->second;
 
 			// handle current fragment
 			if (fragmentNum == incompleteMessage->nextFragmentNum)
@@ -228,8 +220,7 @@ namespace transport
 			{
 				// delete incomplete message
 				incompleteMessage->msg = nullptr;
-				delete incompleteMessage;
+				m_IncompleteMessages.erase (msgID);				
-				m_IncomleteMessages.erase (msgID);				
 				// process message
 				SendMsgAck (msgID);
 				msg->FromSSU (msgID);
@@ -303,10 +294,14 @@ namespace transport
 		}	
 		if (m_SentMessages.empty ()) // schedule resend at first message only
 			ScheduleResend ();
-		SentMessage * sentMessage = new SentMessage;
+		
-		m_SentMessages[msgID] = sentMessage; 
+		auto ret = m_SentMessages.insert (std::make_pair (msgID, std::unique_ptr<SentMessage>(new SentMessage))); 
+		std::unique_ptr<SentMessage>& sentMessage = ret.first->second;
+		if (ret.second)	
+		{
 			sentMessage->nextResendTime = i2p::util::GetSecondsSinceEpoch () + RESEND_INTERVAL;
 			sentMessage->numResends = 0;
+		}	
 		auto& fragments = sentMessage->fragments;
 		size_t payloadSize = m_PacketSize - sizeof (SSUHeader) - 9; // 9  =  flag + #frg(1) + messageID(4) + frag info (3) 
 		size_t len = msg->GetLength ();
@@ -451,7 +446,6 @@ namespace transport
 					else
 					{
 						LogPrint (eLogError, "SSU message has not been ACKed after ", MAX_NUM_RESENDS, " attempts. Deleted");
-						delete it->second;
 						it = m_SentMessages.erase (it);
 					}	
 				}	
@@ -491,13 +485,12 @@ namespace transport
 		if (ecode != boost::asio::error::operation_aborted)
 		{
 			uint32_t ts = i2p::util::GetSecondsSinceEpoch ();
-			for (auto it = m_IncomleteMessages.begin (); it != m_IncomleteMessages.end ();)
+			for (auto it = m_IncompleteMessages.begin (); it != m_IncompleteMessages.end ();)
 			{
 				if (ts > it->second->lastFragmentInsertTime + INCOMPLETE_MESSAGES_CLEANUP_TIMEOUT)
 				{
 					LogPrint (eLogError, "SSU message ", it->first, " was not completed  in ", INCOMPLETE_MESSAGES_CLEANUP_TIMEOUT, " seconds. Deleted");
-					delete it->second;
+					it = m_IncompleteMessages.erase (it);
-					it = m_IncomleteMessages.erase (it);
 				}	
 				else
 					it++;

SSUData.h

@@ -113,8 +113,8 @@ namespace transport
 		private:	
 
 			SSUSession& m_Session;
-			std::map<uint32_t, IncompleteMessage *> m_IncomleteMessages;
+			std::map<uint32_t, std::unique_ptr<IncompleteMessage> > m_IncompleteMessages;
-			std::map<uint32_t, SentMessage *> m_SentMessages;
+			std::map<uint32_t, std::unique_ptr<SentMessage> > m_SentMessages;
 			std::set<uint32_t> m_ReceivedMessages;
 			boost::asio::deadline_timer m_ResendTimer, m_DecayTimer, m_IncompleteMessagesCleanupTimer;
 			int m_MaxPacketSize, m_PacketSize;