storagecommands: drop extra check range condition
The recordType variable is a uint8_t and the <= check will never be
false.
Signed-off-by: Patrick Venture <venture@google.com>
Change-Id: I06a692ad06e9e07fd5e0f170ecf8c7ed27d9d83e
diff --git a/src/storagecommands.cpp b/src/storagecommands.cpp
index 62cd12e..2c96297 100644
--- a/src/storagecommands.cpp
+++ b/src/storagecommands.cpp
@@ -898,8 +898,7 @@
return ipmi::responseSuccess(nextRecordID, recordID, recordType,
oemTsEventType{timestamp, eventData});
}
- else if (recordType >= intel_oem::ipmi::sel::oemEventFirst &&
- recordType <= intel_oem::ipmi::sel::oemEventLast)
+ else if (recordType >= intel_oem::ipmi::sel::oemEventFirst)
{
// Only keep the bytes that fit in the record
std::array<uint8_t, intel_oem::ipmi::sel::oemEventSize> eventData{};