commit 51742323422efe59ff92327d3221b85d78a78d7b
author Junxiao Shi <git@mail1.yoursunny.com> Sun Aug 13 17:04:52 2017 +0000
committer Junxiao Shi <git@mail1.yoursunny.com> Sun Aug 13 17:04:52 2017 +0000
tree f07f5d2337f2e1c2b9e7a9ed4d0a15f4bcd057ad
parent a9181d7bc8321aea1875030e0aab015b06c38ae0

Check integer range during decoding

This commit also moves ControlResponse test suite to the correct
place under mgmt/ instead of mgmt/nfd/.

refs #3200

Change-Id: I1a0578b3e68b8c36c84a6af03d0c47bebe598fe1

src/mgmt/control-response.cpp

diff --git a/src/mgmt/control-response.cpp b/src/mgmt/control-response.cpp
index 9d82d40..3045783 100644
--- a/src/mgmt/control-response.cpp
+++ b/src/mgmt/control-response.cpp
@@ -79,7 +79,7 @@
   if (val == m_wire.elements_end() || val->type() != tlv::nfd::StatusCode) {
     BOOST_THROW_EXCEPTION(Error("missing StatusCode sub-element"));
   }
-  m_code = readNonNegativeInteger(*val);
+  m_code = readNonNegativeIntegerAs<uint32_t>(*val);
   ++val;
 
   if (val == m_wire.elements_end() || val->type() != tlv::nfd::StatusText) {