lib/vty: don't print to closed vty

vty_config_exit() may be called when the vty is already mostly gone,
i.e. when we lost a connection or pipe.  Drop the warnings in that case,
they won't make it anywhere.

Signed-off-by: David Lamparter <equinox@diac24.net>

lib/vty.c

@@ -2615,8 +2615,9 @@ void vty_config_exit(struct vty *vty)
 		node = cnode->parent_node;
 	}
 	if (node != CONFIG_NODE) {
-		vty_out(vty,
-			"WARNING: vty_config_exit() from outside CONFIG_NODE!\n");
+		if (vty->status != VTY_CLOSE)
+			vty_out(vty,
+				"WARNING: vty_config_exit() from outside CONFIG_NODE!\n");
 		return;
 	}
 
@@ -2628,8 +2629,9 @@ void vty_config_exit(struct vty *vty)
 
 	/* Check if there's a pending confirmed commit. */
 	if (vty->t_confirmed_commit_timeout) {
-		vty_out(vty,
-			"WARNING: exiting with a pending confirmed commit. Rolling back to previous configuration.\n\n");
+		if (vty->status != VTY_CLOSE)
+			vty_out(vty,
+				"WARNING: exiting with a pending confirmed commit. Rolling back to previous configuration.\n\n");
 		nb_cli_confirmed_commit_rollback(vty);
 		nb_cli_confirmed_commit_clean(vty);
 	}