commit 0a327e1ce11fb6ef083bffcbb5426e3d10542753
author Suryakanth Sekar <suryakanth.sekar@linux.intel.com> Thu Aug 08 14:30:19 2019 +0530
committer Tom Joseph <tomjoseph@in.ibm.com> Wed Aug 28 09:52:47 2019 +0000
tree 0e7085e0776db45a2e8ad285aa3183740cbaf213
parent e405440f87a4748a7ec116eb3ec44693ede8cf79

Added VLAN ID checking condition in Set LAN config

According to the VLAN 802.1VLAN spec, VLAN ID should be 1-4095

Unit test:
Verified VLAN ID 1-4095 is allowed  and CC returns for 0 & > 4095

Signed-off-by: Suryakanth Sekar <suryakanth.sekar@linux.intel.com>
Change-Id: I54bbc94c814c98dda11f8241fa31bfe0e5bbb150

transporthandler.hpp

diff --git a/transporthandler.hpp b/transporthandler.hpp
index 04d4673..5896082 100644
--- a/transporthandler.hpp
+++ b/transporthandler.hpp
@@ -79,6 +79,8 @@
     IPV6_NEIGHBOR_TIMING_CONFIGURATION = 80,
 };
 
+// Data length of parameters
+constexpr size_t lanParamVLANSize = 4;
 constexpr uint8_t SET_COMPLETE = 0;
 constexpr uint8_t SET_IN_PROGRESS = 1;
 constexpr uint8_t SET_COMMIT_WRITE = 2;         // Optional
@@ -136,3 +138,4 @@
  * @param[in] channel: channel number.
  */
 void applyChanges(int channel);
+constexpr uint16_t maxValidVLANIDValue = 4095;