commit fdb8389df74f9f0d6428252a75c33f6abf6d8481
author Yu Ren <yux.ren@intel.com> Fri May 17 16:47:57 2019 +0800
committer Vernon Mauery <vernon.mauery@linux.intel.com> Mon May 20 17:28:46 2019 +0000
tree 9919eacbe63c66a89e3ae1cfb12906c4b1420085
parent 1554132bc08cf7b22777ae83c3ebd23997a61496

Return error if the input data is larger than expected

If the input data length exceeds the expected, return len_invalid too

Tested:
ipmitool raw 0x06 0x24 0x01 0x13 0x0 0x2 0xa 0x00 0xff

Will return error

Signed-off-by: Yu Ren <yux.ren@intel.com>
Change-Id: I12168f4aed51c499fd06067ca4475917b68e819e

app/watchdog.cpp

diff --git a/app/watchdog.cpp b/app/watchdog.cpp
index b1eea1c..25ca4b1 100644
--- a/app/watchdog.cpp
+++ b/app/watchdog.cpp
@@ -187,7 +187,7 @@
                                  ipmi_context_t context)
 {
     // Extract the request data
-    if (*data_len < sizeof(wd_set_req))
+    if (*data_len != sizeof(wd_set_req))
     {
         *data_len = 0;
         return IPMI_CC_REQ_DATA_LEN_INVALID;