Response with internal error on invalid Chassis path
Added verification of chassis path in case if received path is
invalid. Response with internal error on wrong format of a path.
Tested:
- Verify if chassis collection is filled properly
- Verify if internal error is returned on wrong path
- RedfishServiceValidator.py passes
Signed-off-by: Wludzik, Jozef <jozef.wludzik@intel.com>
Change-Id: Ibe2b9a12d0a5c2c05c9db502b65d2196851352b5
diff --git a/redfish-core/lib/chassis.hpp b/redfish-core/lib/chassis.hpp
index 4e221cd..b0690c8 100644
--- a/redfish-core/lib/chassis.hpp
+++ b/redfish-core/lib/chassis.hpp
@@ -204,7 +204,14 @@
if (lastPos == std::string::npos)
{
BMCWEB_LOG_ERROR << "Failed to find '/' in " << objpath;
- continue;
+ messages::internalError(asyncResp->res);
+ return;
+ }
+ if ((lastPos + 1) >= objpath.size())
+ {
+ BMCWEB_LOG_ERROR << "Failed to parse path " << objpath;
+ messages::internalError(asyncResp->res);
+ return;
}
chassisArray.push_back(
{{"@odata.id", "/redfish/v1/Chassis/" +