Mercurial > vim
changeset 24578:096f2b8d9cf4 v8.2.2828
patch 8.2.2828: Coverity complains about not checking rename() return value
Commit: https://github.com/vim/vim/commit/97a6c6a1fb6043fd6520fbaaafc6970334186167
Author: Bram Moolenaar <Bram@vim.org>
Date: Mon May 3 19:49:51 2021 +0200
patch 8.2.2828: Coverity complains about not checking rename() return value
Problem: Coverity complains about not checking the rename() return value.
Solution: Add "(void)", can't do anything in case of a failure.
author | Bram Moolenaar <Bram@vim.org> |
---|---|
date | Mon, 03 May 2021 20:00:04 +0200 |
parents | 2cf982bc109e |
children | dd22ab1cb899 |
files | src/fileio.c src/version.c |
diffstat | 2 files changed, 3 insertions(+), 1 deletions(-) [+] |
line wrap: on
line diff
--- a/src/fileio.c +++ b/src/fileio.c @@ -3733,7 +3733,7 @@ vim_rename(char_u *from, char_u *to) return 0; // Strange, the second step failed. Try moving the // file back and return failure. - mch_rename(tempname, (char *)from); + (void)mch_rename(tempname, (char *)from); return -1; } // If it fails for one temp name it will most likely fail