Pull request project-chip#2094: [MATTER-4282] [Cherry-pick]Fix off-by-one in look checks for QName iterators. (project-chip#33273)

Merge in WMN_TOOLS/matter from cherry-pick/psirt_wifi to RC_2.3.1-1.3

Squashed commit of the following:

commit 1556eac8f7bb54f90c7f550c61b5d438f0d2123c
Author: Andrei Litvin <andy314@gmail.com>
Date:   Thu May 2 15:06:07 2024 -0400

    Fix off-by-one in look checks for QName iterators. (project-chip#33273)

    Unit test sizes for the string `test` were off by one which
    masked a off-by-one comparison in QName handling.

    Update unit test and comparisons. This will disallow
    backward references to "self" for qnames.

    Co-authored-by: Andrei Litvin <andreilitvin@google.com>

Author: chirag-silabs

src/lib/dnssd/minimal_mdns/core/QName.cpp

@@ -61,7 +61,7 @@ bool SerializedQNameIterator::Next(bool followIndirectPointers)
             }
 
             size_t offset = static_cast<size_t>(((*mCurrentPosition & 0x3F) << 8) | *(mCurrentPosition + 1));
-            if (offset > mLookBehindMax)
+            if (offset >= mLookBehindMax)
             {
                 // Potential infinite recursion.
                 mIsValid = false;

src/lib/dnssd/minimal_mdns/core/tests/TestQName.cpp

@@ -136,7 +136,7 @@ void InvalidReferencing(nlTestSuite * inSuite, void * inContext)
 
     {
         // Infinite recursion
-        static const uint8_t kData[] = "\03test\xc0\x00";
+        static const uint8_t kData[] = "\04test\xc0\x00";
         SerializedQNameIterator it   = AsSerializedQName(kData);
 
         NL_TEST_ASSERT(inSuite, it.Next());
@@ -146,7 +146,7 @@ void InvalidReferencing(nlTestSuite * inSuite, void * inContext)
 
     {
         // Infinite recursion by referencing own element (inside the stream)
-        static const uint8_t kData[] = "\03test\xc0\x05";
+        static const uint8_t kData[] = "\04test\xc0\x05";
         SerializedQNameIterator it   = AsSerializedQName(kData);
 
         NL_TEST_ASSERT(inSuite, it.Next());
@@ -165,7 +165,7 @@ void InvalidReferencing(nlTestSuite * inSuite, void * inContext)
 
     {
         // Reference that goes forwad instead of backward
-        static const uint8_t kData[] = "\03test\xc0\x07";
+        static const uint8_t kData[] = "\04test\xc0\x07";
         SerializedQNameIterator it   = AsSerializedQName(kData);
 
         NL_TEST_ASSERT(inSuite, it.Next());