Skip to content

Commit

Permalink
Fix: FDB flush notification may have no bridge port id attr (sonic-ne…
Browse files Browse the repository at this point in the history
  • Loading branch information
qiluo-msft authored Sep 22, 2018
1 parent d3fa78c commit b9424f0
Showing 1 changed file with 0 additions and 7 deletions.
7 changes: 0 additions & 7 deletions syncd/syncd_notifications.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -107,22 +107,15 @@ void redisPutFdbEntryToAsicView(
{
sai_object_id_t bv_id = fdb->fdb_entry.bv_id;
sai_object_id_t port_oid = 0;
bool port_oid_found = false;

for (uint32_t i = 0; i < fdb->attr_count; i++)
{
if(fdb->attr[i].id == SAI_FDB_ENTRY_ATTR_BRIDGE_PORT_ID)
{
port_oid = fdb->attr[i].value.oid;
port_oid_found = true;
}
}

if (!port_oid_found)
{
SWSS_LOG_ERROR("Failed to get bridge port ID for FDB entry %s",strFdbEntry.c_str());
return;
}

if (!port_oid && !bv_id)
{
Expand Down

0 comments on commit b9424f0

Please sign in to comment.