Merge branch 'hotfix-50.0.1'

Author: rsta2

CHANGELOG.md

@@ -3,9 +3,16 @@ Change Log
 
 This file contains the release notes (the major changes) since Circle Step30 for quick access. For earlier releases please checkout the respective git tag and look into README.md. More info is attached to the release tags (git cat-file tag StepNN) and is available in the git commit log.
 
+Release 50.0.1
+--------------
+
+This is a hotfix release, which fixes a serious bug in the class `CmDNSDaemon`, which also affected the class `CmDNSPublisher`. If you are using one of these classes, you should upgrade.
+
 The 50th Step
 -------------
 
+2025-08-15
+
 This release comes with full (send and receive) network multicast support, including IGMPv2 support. A class `CmDNSDaemon` has been added, which determines and maintains our mDNS hostname on the local network. Name collisions with other hosts will be resolved by appending a numeric suffix to the hostname. The daemon is automatically started, when someone requests a pointer to it. This has been added to the `CmDNSPublisher` class, which works in conjunction with `CmDNSDaemon` now. The *test/mdns-publisher* uses the numeric suffix to generate an unique service name.
 
 More news:

README.md

@@ -8,6 +8,11 @@ Circle is a C++ bare metal programming environment for the Raspberry Pi. It shou
 
 Circle includes bigger (optional) third-party C-libraries for specific purposes in addon/ now. This is the reason why GitHub rates the project as a C-language-project. The main Circle libraries are written in C++ using classes instead. That's why it is called a C++ programming environment.
 
+Release 50.0.1
+--------------
+
+This is a hotfix release, which fixes a serious bug in the class `CmDNSDaemon`, which also affected the class `CmDNSPublisher`. If you are using one of these classes, you should upgrade.
+
 The 50th Step
 -------------
 

Rules.mk

@@ -18,7 +18,7 @@
 # along with this program.  If not, see <http://www.gnu.org/licenses/>.
 #
 
-CIRCLEVER = 500000
+CIRCLEVER = 500001
 
 CIRCLEHOME ?= ..
 

lib/net/mdnsdaemon.cpp

@@ -400,9 +400,11 @@ int CmDNSDaemon::ParseQuery (const u8 *pBuffer, size_t nBufLen)
 	CString Name;
 	u8 uchLen;
 	boolean bFirst = TRUE;
-	while (   nBufLen--
+	while (   nBufLen
 	       && (uchLen = *pMsg++) != 0)
 	{
+		nBufLen--;
+
 		if ((uchLen & 0xC0) == 0xC0)	// compression not allowed here
 		{
 			return -1;
@@ -480,9 +482,11 @@ int CmDNSDaemon::ParseResponse (const u8 *pBuffer, size_t nBufLen, CIPAddress *p
 		CString Name;
 		u8 uchLen;
 		boolean bFirst = TRUE;
-		while (   nBufLen--
+		while (   nBufLen
 		       && (uchLen = *pMsg++) != 0)
 		{
+			nBufLen--;
+
 			if ((uchLen & 0xC0) == 0xC0)	// compression not allowed here
 			{
 				return -1;
@@ -520,7 +524,8 @@ int CmDNSDaemon::ParseResponse (const u8 *pBuffer, size_t nBufLen, CIPAddress *p
 		if (   RRTrailer.nType			== BE (DNS_QTYPE_A)
 		    && (  RRTrailer.nClass
 		        & ~BE (DNS_QCLASS_CACHE_FLUSH))	== BE (DNS_QCLASS_IN)
-		    && RRTrailer.nRDLength		== BE (DNS_RDLENGTH_AIN))
+		    && RRTrailer.nRDLength		== BE (DNS_RDLENGTH_AIN)
+		    && Name.Compare (m_HostnameWithDomain) == 0)
 		{
 			break;
 		}