Correct the calculation of integrity pad length
Change-Id: I34a4ab1adc7a0c510017edea20474008edaf8198
Signed-off-by: Tom Joseph <tomjoseph@in.ibm.com>
diff --git a/message_parsers.cpp b/message_parsers.cpp
index 75c12fa..7b42e00 100644
--- a/message_parsers.cpp
+++ b/message_parsers.cpp
@@ -240,7 +240,7 @@
* .If present each integrity Pad byte is set to FFh. The following logic
* calculates the number of padding bytes added in the IPMI packet.
*/
- auto paddingLen = (4 - ((payloadLen + 2) & 3)) & 3;
+ auto paddingLen = 4 - ((payloadLen + 2) & 3);
auto sessTrailerPos = sizeof(SessionHeader_t) + payloadLen + paddingLen;
@@ -286,7 +286,7 @@
// The following logic calculates the number of padding bytes to be added to
// IPMI packet. If needed each integrity Pad byte is set to FFh.
- auto paddingLen = (4 - ((payloadLen + 2) & 3)) & 3;
+ auto paddingLen = 4 - ((payloadLen + 2) & 3);
packet.insert(packet.end(), paddingLen, 0xFF);
packet.resize(packet.size() + sizeof(SessionTrailer_t));