diff options
| author | Donald Sharp <sharpd@nvidia.com> | 2024-03-12 18:12:48 +0100 |
|---|---|---|
| committer | ton31337 <3352707+ton31337@users.noreply.github.com> | 2024-03-13 08:09:10 +0100 |
| commit | eedbadf606aae5cfd56fa8179bd3991cc47a7fe5 (patch) | |
| tree | d2a2d05214ea7a22543f252d88838fa50d6c8dc8 /mgmtd/mgmt_testc.c | |
| parent | Merge pull request #15539 from donaldsharp/cleanup_leaky_sieve (diff) | |
| download | frr-eedbadf606aae5cfd56fa8179bd3991cc47a7fe5.tar.xz frr-eedbadf606aae5cfd56fa8179bd3991cc47a7fe5.zip | |
eigrpd, mgmtd, ospf6d: frr_fini is last
I noticed that ospf6d always had a linked list memory leak.
Tracking it down shows that frr_fini() shuts down the memory
system and prints out memory not cleaned up. eigrpd, mgmtd
and ospf6d all called cleanup functions after frr_fini leaving
memory leaked that was not really leaked.
Signed-off-by: Donald Sharp <sharpd@nvidia.com>
Diffstat (limited to 'mgmtd/mgmt_testc.c')
| -rw-r--r-- | mgmtd/mgmt_testc.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/mgmtd/mgmt_testc.c b/mgmtd/mgmt_testc.c index 7e3ded820..a33a55efc 100644 --- a/mgmtd/mgmt_testc.c +++ b/mgmtd/mgmt_testc.c @@ -133,8 +133,10 @@ static void sigusr1(void) static void quit(int exit_code) { EVENT_OFF(event_timeout); - frr_fini(); darr_free(__client_cbs.notif_xpaths); + + frr_fini(); + exit(exit_code); } |