vtysh: clean vtysh_file_locked when exiting config node

The flag should be cleared to make sure it's not reused the next time we enter the config node. Signed-off-by: Igor Ryzhov <[email protected]>
FRRouting · Nov 17, 2023 · 40def43 · 40def43
1 parent c599ed5
commit 40def43
Showing 1 changed file with 6 additions and 1 deletion.
diff --git a/vtysh/vtysh.c b/vtysh/vtysh.c
@@ -1652,6 +1652,7 @@ static int vtysh_end(void)
 		/* Nothing to do. */
 		break;
 	default:
+		vty->vtysh_file_locked = false;
 		vty->node = ENABLE_NODE;
 		break;
 	}
@@ -2396,14 +2397,18 @@ static int vtysh_exit(struct vty *vty)
 		vty->node = cnode->parent_node;
 
 	if (vty->node == CONFIG_NODE) {
+		bool locked = vty->vtysh_file_locked;
+
 		/* resync in case one of the daemons is somewhere else */
 		vtysh_execute("end");
 		/* NOTE: a rather expensive thing to do, can we avoid it? */
 
-		if (vty->vtysh_file_locked)
+		if (locked)
 			vtysh_execute("configure terminal file-lock");
 		else
 			vtysh_execute("configure terminal");
+	} else if (vty->node == ENABLE_NODE) {
+		vty->vtysh_file_locked = false;
 	}
 
 	return CMD_SUCCESS;