Commit 5daca042 by Vicent Marti

filebuf: Check the return value for `close`

parent d96c3863
...@@ -319,10 +319,15 @@ int git_filebuf_commit(git_filebuf *file, mode_t mode) ...@@ -319,10 +319,15 @@ int git_filebuf_commit(git_filebuf *file, mode_t mode)
if (verify_last_error(file) < 0) if (verify_last_error(file) < 0)
goto on_error; goto on_error;
p_close(file->fd);
file->fd = -1;
file->fd_is_open = false; file->fd_is_open = false;
if (p_close(file->fd) < 0) {
giterr_set(GITERR_OS, "Failed to close file at '%s'", file->path_lock);
goto on_error;
}
file->fd = -1;
if (p_chmod(file->path_lock, mode)) { if (p_chmod(file->path_lock, mode)) {
giterr_set(GITERR_OS, "Failed to set attributes for file at '%s'", file->path_lock); giterr_set(GITERR_OS, "Failed to set attributes for file at '%s'", file->path_lock);
goto on_error; goto on_error;
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment