CC-tea/coreutils
1
0
Fork 0
mirror of git://git.sv.gnu.org/coreutils.git synced 2026-07-26 02:30:35 +02:00
Code Issues Projects Releases Wiki Activity

(copy_internal): Don't apply the umask in move_mode.

Browse Source 
Otherwise, `mv' would not preserve the permissions when copying
between partitions.  Reported by David Godfrey
This commit is contained in:
Jim Meyering
1999-05-23 23:23:55 +00:00
parent 3957028d28
commit 0e39b529b6
1 changed files with 7 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/copy.c
+7 -1
View File
@@ -896,7 +896,13 @@ copy_internal (const char *src_path, const char *dst_path,
else if ((x->preserve_chmod_bits || new_dst)
&& (x->copy_as_regular || S_ISREG (src_type) || S_ISDIR (src_type)))
{
if (chmod (dst_path, src_mode & x->umask_kill))
mode_t dst_mode = src_mode;
/* Honor the umask for `cp', but not for `mv'. */
if (!x->move_mode)
dst_mode &= x->umask_kill;
if (chmod (dst_path, dst_mode))
{
error (0, errno, _("preserving permissions for %s"), dst_path);
if (x->require_preserve)