From cb463add618b10d5d3445124a9b41d63d147338f Mon Sep 17 00:00:00 2001
From: Buke Po <bukepo@gmail.com>
Date: Wed, 19 Jul 2017 00:03:36 +0800
Subject: [PATCH] [dns] remove functional code from assert() (#2009)

---
 src/core/net/dns_client.cpp | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/core/net/dns_client.cpp b/src/core/net/dns_client.cpp
index 92304d806..b068fde3c 100644
--- a/src/core/net/dns_client.cpp
+++ b/src/core/net/dns_client.cpp
@@ -354,7 +354,8 @@ Message *Client::FindRelatedQuery(const Header &aResponseHeader, QueryMetadata &
     while (message != NULL)
     {
         // Partially read DNS header to obtain message ID only.
-        assert(message->Read(message->GetOffset(), sizeof(messageId), &messageId) == sizeof(messageId));
+        uint16_t count = message->Read(message->GetOffset(), sizeof(messageId), &messageId);
+        assert(count == sizeof(messageId));
 
         if (HostSwap16(messageId) == aResponseHeader.GetMessageId())
         {