commit 38268fa8af4964312062bd92156ab87637f82e71
author Andrew Geissler <geissonator@yahoo.com> Sat May 16 14:22:18 2020 -0500
committer Andrew Geissler <geissonator@yahoo.com> Sat May 16 14:22:18 2020 -0500
tree c3c8d027f296f56bb274179a7d4d23b4e4f06c50
parent 7f2e23e9d4fd2ee6c41811f4d3f2c839fc65e8d0

vlan id should be 32 bit

Upstream yocto appears to bring in a new compiler which is flagging the
inconsistency of the VLAN ID data type and failing to compile bmcweb
because of it.

The redfish specification lists the datatype for VLANId as a integer
that can range from 0-4094.

Here's the error:
/lib/ethernet.hpp:2221:58: error: no matching function for call to 'std::variant<unsigned int>::variant(uint64_t&)'
|  2221 |                             std::variant<uint32_t>(vlanId));

Change-Id: Ic92ad6518a9529f5099a1eaf1250df106119d319
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>

redfish-core/lib/ethernet.hpp

diff --git a/redfish-core/lib/ethernet.hpp b/redfish-core/lib/ethernet.hpp
index 04759fe..1f65254 100644
--- a/redfish-core/lib/ethernet.hpp
+++ b/redfish-core/lib/ethernet.hpp
@@ -2187,7 +2187,7 @@
         }
 
         bool vlanEnable = false;
-        uint64_t vlanId = 0;
+        uint32_t vlanId = 0;
 
         if (!json_util::readJson(req, res, "VLANEnable", vlanEnable, "VLANId",
                                  vlanId))