From 88d946cc08e1f692fb3cf3063e6f659345117c68 Mon Sep 17 00:00:00 2001
From: Abtin Keshavarzian <abtink@google.com>
Date: Mon, 15 Nov 2021 10:17:58 -0800
Subject: [PATCH] [srp-server] free heap allocated data from `Free()` (#7165)

This commit fixes the potential memory leak of `mTxtData` when a
`Description` instance is freed. Also addresses similar situation
with `mServiceName` when `Service` instance is freed.
---
 src/core/net/srp_server.cpp | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/src/core/net/srp_server.cpp b/src/core/net/srp_server.cpp
index dda9f844b..190fb579c 100644
--- a/src/core/net/srp_server.cpp
+++ b/src/core/net/srp_server.cpp
@@ -1469,6 +1469,7 @@ exit:
 
 void Server::Service::Free(void)
 {
+    mServiceName.Free();
     Heap::Free(this);
 }
 
@@ -1619,6 +1620,7 @@ exit:
 void Server::Service::Description::Free(void)
 {
     mInstanceName.Free();
+    Heap::Free(mTxtData);
     Heap::Free(this);
 }