proxmox-mirrors/mirror_frr
1
0
Fork 0
mirror of https://git.proxmox.com/git/mirror_frr synced 2025-08-13 22:57:45 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Reorder free(f); unlink(f); to unlink before freeing.

Browse Source
This commit is contained in:
gdt 2003-12-22 20:18:18 +00:00
parent aa593d5e26
commit 3dbf99698a
2 changed files with 7 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
lib/ChangeLog
View File

@ -1,3 +1,8 @@
2003-12-22 Greg Troxel <gdt@fnord.ir.bbn.com>
* vty.c (vty_use_backup_config): Don't free filenames before using
them for unlink.
2003-08-20 Yasuhiro Ohara <yasu@sfc.wide.ad.jp> 2003-08-20 Yasuhiro Ohara <yasu@sfc.wide.ad.jp>
* command.c: Fix <cr> display problem for command line * command.c: Fix <cr> display problem for command line

4
lib/vty.c
View File

@ -2173,9 +2173,9 @@ vty_use_backup_config (char *fullpath)
sav = open (fullpath_sav, O_RDONLY); sav = open (fullpath_sav, O_RDONLY);
if (sav < 0) if (sav < 0)
{ {
unlink (fullpath_tmp);
free (fullpath_sav); free (fullpath_sav);
free (fullpath_tmp); free (fullpath_tmp);
unlink (fullpath_tmp);
return NULL; return NULL;
} }
@ -2187,9 +2187,9 @@ vty_use_backup_config (char *fullpath)
if (chmod(fullpath_tmp, CONFIGFILE_MASK) != 0) if (chmod(fullpath_tmp, CONFIGFILE_MASK) != 0)
{ {
unlink (fullpath_tmp);
free (fullpath_sav); free (fullpath_sav);
free (fullpath_tmp); free (fullpath_tmp);
unlink (fullpath_tmp);
return NULL; return NULL;
} }