KurlesHS/restic
1
0
Fork 0
forked from TrueCloudLab/restic
Code Pull requests Activity

local/sftp: Remove unneeded stat() call

Browse source
This commit is contained in:
Alexander Neumann 2017-07-20 22:17:47 +02:00
parent c40b3d3983
commit 9842eff887
4 changed files with 5 additions and 18 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
internal/backend/local/local.go
View file

@ -149,13 +149,7 @@ func (b *Local) Save(ctx context.Context, h restic.Handle, rd io.Reader) (err er
return errors.Wrap(err, "Close") return errors.Wrap(err, "Close")
} }
// set mode to read-only return setNewFileMode(filename, backend.Modes.File)
fi, err := fs.Stat(filename)
if err != nil {
return errors.Wrap(err, "Stat")
}
return setNewFileMode(filename, fi)
} }
// Load returns a reader that yields the contents of the file at h at the // Load returns a reader that yields the contents of the file at h at the

4
internal/backend/local/local_unix.go
View file

@ -9,6 +9,6 @@ import (
) )
// set file to readonly // set file to readonly
func setNewFileMode(f string, fi os.FileInfo) error { func setNewFileMode(f string, mode os.FileMode) error {
return fs.Chmod(f, fi.Mode()&os.FileMode(^uint32(0222))) return fs.Chmod(f, mode)
} }

2
internal/backend/local/local_windows.go
View file

@ -7,6 +7,6 @@ import (
// We don't modify read-only on windows, // We don't modify read-only on windows,
// since it will make us unable to delete the file, // since it will make us unable to delete the file,
// and this isn't common practice on this platform. // and this isn't common practice on this platform.
func setNewFileMode(f string, fi os.FileInfo) error { func setNewFileMode(f string, mode os.FileMode) error {
return nil return nil
} }

9
internal/backend/sftp/sftp.go
View file

@ -348,14 +348,7 @@ func (r *SFTP) Save(ctx context.Context, h restic.Handle, rd io.Reader) (err err
return errors.Wrap(err, "Close") return errors.Wrap(err, "Close")
} }
// set mode to read-only return errors.Wrap(r.c.Chmod(filename, backend.Modes.File), "Chmod")
fi, err := r.c.Lstat(filename)
if err != nil {
return errors.Wrap(err, "Lstat")
}
err = r.c.Chmod(filename, fi.Mode()&os.FileMode(^uint32(0222)))
return errors.Wrap(err, "Chmod")
} }
// Load returns a reader that yields the contents of the file at h at the // Load returns a reader that yields the contents of the file at h at the