Skip to content

Commit 57dccdb

Browse files
committed
ripd: remove vty configuration lock
The vty configuration lock is used to prevent inconsistencies when multiple users are editing the configuration at the same time. The pointer stored in vty->index might become invalid if the associated configuration object is removed by another user in another CLI session. Commands converted to the new northbound model don't use vty->index, but vty->xpath_index and the vty->xpath array. The nb_cli_cfg_change() function uses the VTY_CHECK_XPATH macro to check if the configuration object being edited still exists and returns an error if it doesn't. Now that all ripd commands were converted to the new northbound model, remove the ripd vty lock because it's not necessary anymore. Signed-off-by: Renato Westphal <[email protected]>
1 parent 5e57edb commit 57dccdb

File tree

1 file changed

+2
-0
lines changed

1 file changed

+2
-0
lines changed

ripd/rip_main.c

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -170,6 +170,8 @@ int main(int argc, char **argv)
170170
}
171171
}
172172

173+
vty_config_lockless();
174+
173175
/* Prepare master thread. */
174176
master = frr_init();
175177

0 commit comments

Comments
 (0)