From 3753761963b8b550220bfb0eb5e8ad8d088e2b47 Mon Sep 17 00:00:00 2001 From: Ember Date: Sun, 5 Apr 2026 17:48:42 -0700 Subject: [PATCH] Fix memory leak of m_configData in CDMRNetwork m_configData was allocated in setConfig() but never freed in the destructor. Also guard against leak on repeated setConfig() calls by freeing the previous allocation first. --- DMRNetwork.cpp | 2 ++ 1 file changed, 2 insertions(+) diff --git a/DMRNetwork.cpp b/DMRNetwork.cpp index 290f613..140296f 100644 --- a/DMRNetwork.cpp +++ b/DMRNetwork.cpp @@ -78,6 +78,7 @@ CDMRNetwork::~CDMRNetwork() delete[] m_buffer; delete[] m_salt; delete[] m_id; + delete[] m_configData; } void CDMRNetwork::setOptions(const std::string& options) @@ -87,6 +88,7 @@ void CDMRNetwork::setOptions(const std::string& options) void CDMRNetwork::setConfig(const unsigned char* data, unsigned int len) { + delete[] m_configData; m_configData = new unsigned char[len]; ::memcpy(m_configData, data, len);