mirrors/bird
mirrors/bird
0
0
Fork 0
mirror of https://gitlab.nic.cz/labs/bird.git synced 2024-12-22 09:41:54 +00:00
Code Releases Activity

Conf: Fix too early free of old configuration

Browse Source 
The change 371eb49043 introduced early free
of old_config. Unfortunately, it did not properly check whether it is not
still in use (blocked by obstacle during reconfiguration). Fix that.

It also means that we still could have a short peak when three configs
are in use (when a new reconfig is requeste while the previous one is
still active).
This commit is contained in:
Ondrej Zajicek 2023-02-19 03:59:10 +01:00 committed by Jakub Ružička
parent 132b3493bd
commit 0443fbbe17
1 changed files with 10 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
conf/conf.c
View File

@ -197,8 +197,12 @@ cleanup:
void
config_free(struct config *c)
{
if (c)
rfree(c->pool);
if (!c)
return;
ASSERT(!c->obstacle_count);
rfree(c->pool);
}
/**
@ -207,10 +211,14 @@ config_free(struct config *c)
* This function frees the old configuration (%old_config) that is saved for the
* purpose of undo. It is useful before parsing a new config when reconfig is
* requested, to avoid keeping three (perhaps memory-heavy) configs together.
* Configuration is not freed when it is still active during reconfiguration.
*/
void
config_free_old(void)
{
if (!old_config || old_config->obstacle_count)
return;
tm_stop(config_timer);
undo_available = 0;