diff options
| author | paco <paco@voltanet.io> | 2018-06-21 12:09:44 +0200 |
|---|---|---|
| committer | paco <paco@voltanet.io> | 2018-06-22 00:01:42 +0200 |
| commit | 1a40fad5688f1582371627089a23e697ac3b68f3 (patch) | |
| tree | 23144148189d1f57f8d26e834c96f9195508dbe8 | |
| parent | 76cd18c9ddfbdb8f36b60fac61ebb4d64d12d079 (diff) | |
vtysh: return value check (Coverity 1399200)
Signed-off-by: F. Aragon <paco@voltanet.io>
| -rw-r--r-- | vtysh/vtysh.c | 9 |
1 files 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); } |