[ip6] remove extra param from Ip6::HandleExtensionHeaders() (#8035)

This commit updates `Ip6::HandleExtensionHeaders()` removing
`aIsOutbound` input and instead using `(aNetif == nullptr)` to
determine this.
This commit is contained in:
Abtin Keshavarzian
2022-08-17 14:32:18 -07:00
committed by Jonathan Hui
parent 7f73cd42a4
commit 2d4896f675
2 changed files with 6 additions and 7 deletions
+6 -6
View File
@@ -890,11 +890,11 @@ Error Ip6::HandleExtensionHeaders(Message & aMessage,
MessageInfo &aMessageInfo,
Header & aHeader,
uint8_t & aNextHeader,
bool aIsOutbound,
bool aFromHost,
bool & aReceive)
{
Error error = kErrorNone;
Error error = kErrorNone;
bool isOutbound = (aNetif == nullptr);
ExtensionHeader extHeader;
while (aReceive || aNextHeader == kProtoHopOpts)
@@ -904,7 +904,7 @@ Error Ip6::HandleExtensionHeaders(Message & aMessage,
switch (aNextHeader)
{
case kProtoHopOpts:
SuccessOrExit(error = HandleOptions(aMessage, aHeader, aIsOutbound, aReceive));
SuccessOrExit(error = HandleOptions(aMessage, aHeader, isOutbound, aReceive));
break;
case kProtoFragment:
@@ -916,7 +916,7 @@ Error Ip6::HandleExtensionHeaders(Message & aMessage,
break;
case kProtoDstOpts:
SuccessOrExit(error = HandleOptions(aMessage, aHeader, aIsOutbound, aReceive));
SuccessOrExit(error = HandleOptions(aMessage, aHeader, isOutbound, aReceive));
break;
case kProtoIp6:
@@ -1213,8 +1213,8 @@ start:
// process IPv6 Extension Headers
nextHeader = static_cast<uint8_t>(header.GetNextHeader());
SuccessOrExit(error = HandleExtensionHeaders(aMessage, aNetif, messageInfo, header, nextHeader, aNetif == nullptr,
aFromHost, receive));
SuccessOrExit(error =
HandleExtensionHeaders(aMessage, aNetif, messageInfo, header, nextHeader, aFromHost, receive));
// process IPv6 Payload
if (receive)
-1
View File
@@ -333,7 +333,6 @@ private:
MessageInfo &aMessageInfo,
Header & aHeader,
uint8_t & aNextHeader,
bool aIsOutbound,
bool aFromHost,
bool & aReceive);
Error FragmentDatagram(Message &aMessage, uint8_t aIpProto);