Fixes DNS Server Memory Leak when deleted (espressif#6707)

SuGlider · web-flow · commit 44beee2f2cac · 2022-05-09T15:59:24.000+03:00
diff --git a/libraries/DNSServer/src/DNSServer.cpp b/libraries/DNSServer/src/DNSServer.cpp
@@ -20,6 +20,22 @@ DNSServer::DNSServer()
   _port = 0;
 }
 
+DNSServer::~DNSServer()
+{
+  if (_dnsHeader) {
+    free(_dnsHeader);
+    _dnsHeader = NULL;
+  }
+  if (_dnsQuestion) {
+    free(_dnsQuestion);
+    _dnsQuestion = NULL;
+  }
+  if (_buffer) {
+    free(_buffer);
+    _buffer = NULL;
+  }
+}
+
 bool DNSServer::start(const uint16_t &port, const String &domainName,
                      const IPAddress &resolvedIP)
 {
diff --git a/libraries/DNSServer/src/DNSServer.h b/libraries/DNSServer/src/DNSServer.h
@@ -76,6 +76,7 @@ class DNSServer
 {
   public:
     DNSServer();
+    ~DNSServer();
     void processNextRequest();
     void setErrorReplyCode(const DNSReplyCode &replyCode);
     void setTTL(const uint32_t &ttl);

Original file line number	Diff line number	Diff line change
`@@ -76,6 +76,7 @@ class DNSServer`
`76`	`76`	`{`
`77`	`77`	`public:`
`78`	`78`	`DNSServer();`
	`79`	`+ ~DNSServer();`
`79`	`80`	`void processNextRequest();`
`80`	`81`	`void setErrorReplyCode(const DNSReplyCode &replyCode);`
`81`	`82`	`void setTTL(const uint32_t &ttl);`