espressif/openthread
1
0
Fork 0
mirror of https://github.com/espressif/openthread.git synced 2026-10-18 05:28:10 +00:00
Code Issues Packages Projects Releases Wiki Activity

[mac] don't append header termination IE2 for empty data payload (#7123)

Browse Source 
We do not require the HT2 IE if the only thing that follows the header
is the MIC.

This commit reverts #6941.
This commit is contained in:
Łukasz Maciejończyk
2021-10-29 10:56:55 -07:00
committed by GitHub
parent c9d68a7a01
commit 319068637c
1 changed files with 2 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/core/thread/mesh_forwarder.cpp
+2 -4
View File
@@ -1582,10 +1582,8 @@ void MeshForwarder::AppendHeaderIe(const Message *aMessage, Mac::TxFrame &aFrame
{ {
uint8_t index = 0; uint8_t index = 0;
bool iePresent = false; bool iePresent = false;
// MIC is a part of Data Payload, so if it's present, Data Payload is not empty even if the message is bool payloadPresent =
// MIC is always present when the frame is secured (aFrame.GetType() == Mac::Frame::kFcfFrameMacCmd) || (aMessage != nullptr && aMessage->GetLength() != 0);
bool payloadPresent = (aFrame.GetType() == Mac::Frame::kFcfFrameMacCmd) ||
(aMessage != nullptr && aMessage->GetLength() != 0) || aFrame.GetSecurityEnabled();
#if OPENTHREAD_CONFIG_TIME_SYNC_ENABLE #if OPENTHREAD_CONFIG_TIME_SYNC_ENABLE
if (aMessage != nullptr && aMessage->IsTimeSync()) if (aMessage != nullptr && aMessage->IsTimeSync())