From 993f537a03273e69ade90a38a653e367cb1a6c30 Mon Sep 17 00:00:00 2001
From: Jonathan Hui <jonhui@nestlabs.com>
Date: Fri, 25 Jan 2019 08:35:44 -0800
Subject: [PATCH] [cli] add ping interval bounds check to avoid Timer::Start()
 assert (#3507)

---
 src/cli/cli.cpp | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/src/cli/cli.cpp b/src/cli/cli.cpp
index e55bf4e9d..7e816b2b1 100644
--- a/src/cli/cli.cpp
+++ b/src/cli/cli.cpp
@@ -1923,15 +1923,16 @@ void Interpreter::ProcessPing(int argc, char *argv[])
         switch (index)
         {
         case 1:
-            mLength = (uint16_t)value;
+            mLength = static_cast<uint16_t>(value);
             break;
 
         case 2:
-            mCount = (uint16_t)value;
+            mCount = static_cast<uint16_t>(value);
             break;
 
         case 3:
-            mInterval = (uint32_t)value;
+            mInterval = static_cast<uint32_t>(value);
+            VerifyOrExit(mInterval <= Timer::kMaxDt, error = OT_ERROR_INVALID_ARGS);
             mInterval = mInterval * 1000;
             break;