Skip to content

Commit

Permalink
[routesync] Stale neighbor fix review comment fix
Browse files Browse the repository at this point in the history
Changes done to stale neighbor fix PR#2553 (sonic-net#2553)
Changes include adding check to delete the stale neighbor from ASIC_DB
and APPL_DB only if the kernel command is RTM_DELROUTE

Signed-off-by: vedganes <veda.ganesan@nokia.com>
  • Loading branch information
vganesan-nokia committed Jan 5, 2024
1 parent 8fa982d commit 23ffdcb
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion fpmsyncd/routesync.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -782,7 +782,7 @@ void RouteSync::onRouteMsg(int nlmsg_type, struct nl_object *obj, char *vrf)
// In this case since we do not want the route with next hop on eth0/docker0, we return.
// But still we need to clear the route from the APPL_DB. Otherwise the APPL_DB and data
// path will be left with stale route entry
if(alsv.size() == 1)
if((nlmsg_type == RTM_DELROUTE) && (alsv.size() == 1))
{
if (!warmRestartInProgress)
{
Expand Down

0 comments on commit 23ffdcb

Please sign in to comment.