Updated as per new review feedback and fixed builds

Author: su-shanka

src/lib/dnssd/Discovery_ImplPlatform.cpp

@@ -96,8 +96,7 @@ static void HandleNodeBrowse(void * context, DnssdService * services, size_t ser
         auto & ipAddress = services[i].mAddress;
 
         // mType(service name) exactly matches with operational service name
-        bool isOperationalBrowse = (strncmp(services[i].mType, kOperationalServiceName, sizeof(services[i].mType)) == 0 &&
-                                    strlen(services[i].mType) == strlen(kOperationalServiceName));
+        bool isOperationalBrowse = (strcmp(services[i].mType, kOperationalServiceName, sizeof(services[i].mType)) == 0);
 
         // For operational browse result we currently don't need IP address hence skip resolution and handle differently.
         if (isOperationalBrowse)

src/platform/Darwin/DnssdContexts.cpp

@@ -609,8 +609,7 @@ bool ResolveContext::TryReportingResultsForInterfaceIndex(uint32_t interfaceInde
         DiscoveredNodeData nodeData;
 
         // mType(service name) exactly matches with operational service name
-        if (strncmp(services[i].mType, kOperationalServiceName, sizeof(services[i].mType)) == 0 &&
-            strlen(services[i].mType) == strlen(kOperationalServiceName))
+        if (strncmp(service[i].mType, kOperationalServiceName, sizeof(service[i].mType)) == 0)
         {
             service.ToDiscoveredOperationalNodeBrowseData(nodeData);
         }