Skip to content

Commit 3daaa8e

Browse files
authored
[macsec] Fix acl entry id in updateMACsecPort (#1783)
* Fix acl entry id in updateMACsecPort * Extra fix to calling setMACsecFlowActive
1 parent ec96602 commit 3daaa8e

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

orchagent/macsecorch.cpp

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1106,13 +1106,13 @@ bool MACsecOrch::updateMACsecPort(MACsecPort &macsec_port, const TaskArgs &port_
11061106
SWSS_LOG_WARN("Cannot change the ACL entry action from packet action to MACsec flow");
11071107
return false;
11081108
}
1109-
auto an = macsec_sc->m_encoding_an;
1109+
auto entry_id = macsec_sc->m_entry_id;
11101110
auto flow_id = macsec_sc->m_flow_id;
1111-
recover.add_action([this, an, flow_id]() { this->setMACsecFlowActive(an, flow_id, false); });
1111+
recover.add_action([this, entry_id, flow_id]() { this->setMACsecFlowActive(entry_id, flow_id, false); });
11121112
}
11131113
else
11141114
{
1115-
setMACsecFlowActive(macsec_sc->m_encoding_an, macsec_sc->m_flow_id, false);
1115+
setMACsecFlowActive(macsec_sc->m_entry_id, macsec_sc->m_flow_id, false);
11161116
}
11171117
}
11181118
}
@@ -1696,7 +1696,7 @@ task_process_status MACsecOrch::createMACsecSA(
16961696
}
16971697
recover.add_action([this, sc]() {
16981698
this->setMACsecFlowActive(
1699-
sc->m_encoding_an,
1699+
sc->m_entry_id,
17001700
sc->m_flow_id,
17011701
false);
17021702
});

0 commit comments

Comments
 (0)