diff options
| author | Eric Kinzie <ekinzie@labn.net> | 2022-04-01 00:47:30 +0000 |
|---|---|---|
| committer | Louis Scalbert <louis.scalbert@6wind.com> | 2023-04-18 11:33:15 +0200 |
| commit | 224eadd955b37d6eee501cecf85498eac3ad8d5b (patch) | |
| tree | 9bc05115339a12ee9bf1b708d2dda01f0cc7cdd2 /isisd/isis_route.c | |
| parent | 80ef2e89eeba4924cdd712535cf87e2a7ef4a49f (diff) | |
isisd: fix merging flex-algo route nodes
As isis_route_verify_merge() builds a new route table, each (src,dst)
new node in the table is created by merging data from the same (src,dst)
pair present in all route tables. However, as it iterates over those
route nodes and copies SR data into the newly created node, it repeatedly
overwrites the isis_route_info pointer. This, in turn, pollutes the
isis_route_info structures in all of the other tables with partially
updated SR data.
After a route is deleted, this leaves pointers to freed memory in one
of the route trees.
Fixes: 26b0598f6b6f ("isisd: fix isis_route_merge_verify logic")
Signed-off-by: Eric Kinzie <ekinzie@labn.net>
Diffstat (limited to 'isisd/isis_route.c')
| -rw-r--r-- | isisd/isis_route.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/isisd/isis_route.c b/isisd/isis_route.c index 3983c704ce..be92dcc22e 100644 --- a/isisd/isis_route.c +++ b/isisd/isis_route.c @@ -889,8 +889,9 @@ static void _isis_route_verify_merge(struct isis_area *area, ISIS_ROUTE_FLAG_ZEBRA_SYNCED)) { continue; } + } else { + mrnode->info = rnode->info; } - mrnode->info = rnode->info; } } |