espressif/openthread
1
0
Fork 0
mirror of https://github.com/espressif/openthread.git synced 2026-08-14 22:57:46 +00:00
Code Issues Packages Projects Releases Wiki Activity

[router-table] signal if router is removed from UpdateRouterIdSet() (#8573)

Browse Source 
This commit updates the `RouterTable::RemoveRouter()` to check
if `Router` being removed is a neighbor and signal its removal.
This ensures that we correctly signal neighboring router removal
from a call to either `UpdateRouterIdSet()` or `Release(aRouterId)`.
This commit is contained in:
Abtin Keshavarzian
2022-12-22 10:41:08 -08:00
committed by GitHub
parent ddde5fa5d6
commit 289d24e06e
1 changed files with 5 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/core/thread/router_table.cpp
+5 -5
View File
@@ -101,6 +101,11 @@ void RouterTable::RemoveRouter(Router &aRouter)
// Remove an existing `aRouter` entry from `mRouters` and update the
// `mRouterIdMap`.
if (aRouter.IsStateValid())
{
Get<NeighborTable>().Signal(NeighborTable::kRouterRemoved, aRouter);
}
mRouterIdMap.Release(aRouter.GetRouterId());
mRouters.Remove(aRouter);
@@ -186,11 +191,6 @@ Error RouterTable::Release(uint8_t aRouterId)
router = FindRouterById(aRouterId);
VerifyOrExit(router != nullptr, error = kErrorNotFound);
if (router->IsStateValid())
{
Get<NeighborTable>().Signal(NeighborTable::kRouterRemoved, *router);
}
RemoveRouter(*router);
for (Router &otherRouter : mRouters)