Skip to content

Commit

Permalink
Create ACL table fails due to incorrect check for supported ACL actions
Browse files Browse the repository at this point in the history
sonic-net#11235 (sonic-net#2351)

Signed-off-by: rck-innovium <rck@innovium.com>
  • Loading branch information
rck-innovium committed Jun 24, 2022
1 parent 1ed0b4b commit 1b8bd94
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions orchagent/aclorch.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -153,7 +153,7 @@ static const acl_capabilities_t defaultAclActionsSupported =
}
};

static acl_table_action_list_lookup_t defaultAclActionList =
static acl_table_action_list_lookup_t defaultAclActionList =
{
{
// L3
Expand Down Expand Up @@ -326,7 +326,7 @@ static acl_table_action_list_lookup_t defaultAclActionList =
// The match fields for certain ACL table type are not exactly the same between INGRESS and EGRESS.
// For example, we can only match IN_PORT for PFCWD table type at INGRESS.
// Hence we need to specify stage particular matching fields in stageMandatoryMatchFields
static acl_table_match_field_lookup_t stageMandatoryMatchFields =
static acl_table_match_field_lookup_t stageMandatoryMatchFields =
{
{
// TABLE_TYPE_PFCWD
Expand Down Expand Up @@ -2045,7 +2045,7 @@ bool AclTable::addMandatoryActions()
// Add the default action list
for (auto action : defaultAclActionList[type.getName()][stage])
{
if (m_pAclOrch->isAclActionSupported(stage, acl_action))
if (m_pAclOrch->isAclActionSupported(stage, action))
{
SWSS_LOG_INFO("Added default action for table type %s stage %s",
type.getName().c_str(),
Expand Down

0 comments on commit 1b8bd94

Please sign in to comment.