[portsorch] Fix inconsistent return value in bindAclTable (#791)

Signed-off-by: yorke <yorke.yuan@asterfusion.com>
sonic-net · Mar 21, 2019 · ea54825 · ea54825
1 parent 5984e3a
commit ea54825
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/orchagent/portsorch.cpp b/orchagent/portsorch.cpp
@@ -826,7 +826,7 @@ bool PortsOrch::createBindAclTableGroup(sai_object_id_t id, sai_object_id_t &gro
  {
  SWSS_LOG_ERROR("Failed to bind port %s to ACL table group %lx, rv:%d",
  port.m_alias.c_str(), group_oid, status);
- return status;
+ return false;
  }
  break;
  }
@@ -842,7 +842,7 @@ bool PortsOrch::createBindAclTableGroup(sai_object_id_t id, sai_object_id_t &gro
  {
  SWSS_LOG_ERROR("Failed to bind LAG %s to ACL table group %lx, rv:%d",
  port.m_alias.c_str(), group_oid, status);
- return status;
+ return false;
  }
  break;
  }
@@ -858,14 +858,14 @@ bool PortsOrch::createBindAclTableGroup(sai_object_id_t id, sai_object_id_t &gro
  {
  SWSS_LOG_ERROR("Failed to bind VLAN %s to ACL table group %lx, rv:%d",
  port.m_alias.c_str(), group_oid, status);
- return status;
+ return false;
  }
  break;
  }
  default:
  {
  SWSS_LOG_ERROR("Failed to bind %s port with type %d", port.m_alias.c_str(), port.m_type);
- return SAI_STATUS_FAILURE;
+ return false;
  }
  }