diff options
Diffstat (limited to 'include/ipmid/message.hpp')
-rw-r--r-- | include/ipmid/message.hpp | 10 |
1 files changed, 4 insertions, 6 deletions
diff --git a/include/ipmid/message.hpp b/include/ipmid/message.hpp index 7f1ad7e..b10fc42 100644 --- a/include/ipmid/message.hpp +++ b/include/ipmid/message.hpp @@ -103,15 +103,14 @@ struct Payload Payload(Payload&&) = default; Payload& operator=(Payload&&) = default; - explicit Payload(std::vector<uint8_t>&& data) : - raw(std::move(data)), unpackCheck(false) + explicit Payload(std::vector<uint8_t>&& data) : raw(std::move(data)) { } ~Payload() { using namespace phosphor::logging; - if (trailingOk && !unpackCheck && !fullyUnpacked()) + if (raw.size() != 0 && !trailingOk && !unpackCheck && !unpackError) { log<level::ERR>("Failed to check request for full unpack"); } @@ -449,8 +448,8 @@ struct Payload size_t bitCount = 0; std::vector<uint8_t> raw; size_t rawIndex = 0; - bool trailingOk = false; - bool unpackCheck = true; + bool trailingOk = true; + bool unpackCheck = false; bool unpackError = false; }; @@ -573,7 +572,6 @@ struct Request // not all bits were consumed by requested parameters return ipmi::ccReqDataLenInvalid; } - payload.unpackCheck = false; } } return unpackRet; |