From 1a40fad5688f1582371627089a23e697ac3b68f3 Mon Sep 17 00:00:00 2001 From: paco Date: Thu, 21 Jun 2018 12:09:44 +0200 Subject: [PATCH] vtysh: return value check (Coverity 1399200) Signed-off-by: F. Aragon --- vtysh/vtysh.c | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/vtysh/vtysh.c b/vtysh/vtysh.c index 309493b13e..a64e568daa 100644 --- a/vtysh/vtysh.c +++ b/vtysh/vtysh.c @@ -2634,8 +2634,13 @@ static void backup_config_file(const char *fbackup) strcat(integrate_sav, CONF_BACKUP_EXT); /* Move current configuration file to backup config file. */ - unlink(integrate_sav); - rename(fbackup, integrate_sav); + if (unlink(integrate_sav) != 0) { + vty_out(vty, "Warning: %s unlink failed\n", integrate_sav); + } + if (rename(fbackup, integrate_sav) != 0) { + vty_out(vty, "Error renaming %s to %s\n", fbackup, + integrate_sav); + } free(integrate_sav); } -- 2.39.5