Mercurial > vim
diff src/os_unix.c @ 12847:14f287552218 v8.0.1300
patch 8.0.1300: file permissions may end up wrong when writing
commit https://github.com/vim/vim/commit/cd142e3369db8888163a511dbe9907bcd138829c
Author: Bram Moolenaar <Bram@vim.org>
Date: Thu Nov 16 17:03:45 2017 +0100
patch 8.0.1300: file permissions may end up wrong when writing
Problem: File permissions may end up wrong when writing.
Solution: Use fchmod() instead of chmod() when possible. Don't truncate
until we know we can change the file.
author | Christian Brabandt <cb@256bit.org> |
---|---|
date | Thu, 16 Nov 2017 17:15:06 +0100 |
parents | 351cf7c67bbe |
children | ebb4f6c93598 |
line wrap: on
line diff
--- a/src/os_unix.c +++ b/src/os_unix.c @@ -2725,9 +2725,8 @@ mch_getperm(char_u *name) } /* - * set file permission for 'name' to 'perm' - * - * return FAIL for failure, OK otherwise + * Set file permission for "name" to "perm". + * Return FAIL for failure, OK otherwise. */ int mch_setperm(char_u *name, long perm) @@ -2741,6 +2740,18 @@ mch_setperm(char_u *name, long perm) (mode_t)perm) == 0 ? OK : FAIL); } +#if defined(HAVE_FCHMOD) || defined(PROTO) +/* + * Set file permission for open file "fd" to "perm". + * Return FAIL for failure, OK otherwise. + */ + int +mch_fsetperm(int fd, long perm) +{ + return (fchmod(fd, (mode_t)perm) == 0 ? OK : FAIL); +} +#endif + #if defined(HAVE_ACL) || defined(PROTO) # ifdef HAVE_SYS_ACL_H # include <sys/acl.h>