octoleo/restic
2
2
Fork 0
mirror of https://github.com/octoleo/restic.git synced 2024-11-27 07:16:40 +00:00
Code Issues Releases Wiki Activity

local/sftp: Fix broken error handling

Browse Source 
This yields the error messages for a full backup location:

    panic: write /home/fd0/mnt/temp/tmp/temp-987810174: no space left on device

Closes #540

Also connected to #574
This commit is contained in:
Alexander Neumann 2016-08-28 12:34:53 +02:00
parent 93e62c6f18
commit 196bbbd25b
2 changed files with 11 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
src/restic/backend/local/local.go
View File

@ -176,6 +176,9 @@ func (b *Local) Save(h backend.Handle, p []byte) (err error) {
tmpfile, err := writeToTempfile(filepath.Join(b.p, backend.Paths.Temp), p)
debug.Log("local.Save", "saved %v (%d bytes) to %v", h, len(p), tmpfile)
if err != nil {
return err
}
filename := filename(b.p, h.Type, h.Name)

13
src/restic/backend/sftp/sftp.go
View File

@ -5,7 +5,6 @@ import (
"encoding/hex"
"fmt"
"io"
"log"
"os"
"os/exec"
"path"
@ -44,22 +43,22 @@ func startClient(program string, args ...string) (*SFTP, error) {
// get stdin and stdout
wr, err := cmd.StdinPipe()
if err != nil {
log.Fatal(err)
return nil, err
}
rd, err := cmd.StdoutPipe()
if err != nil {
log.Fatal(err)
return nil, err
}
// start the process
if err := cmd.Start(); err != nil {
log.Fatal(err)
return nil, err
}
// open the SFTP session
client, err := sftp.NewClientPipe(rd, wr)
if err != nil {
log.Fatal(err)
return nil, err
}
return &SFTP{c: client, cmd: cmd}, nil
@ -329,6 +328,10 @@ func (r *SFTP) Save(h backend.Handle, p []byte) (err error) {
}
filename, tmpfile, err := r.tempFile()
if err != nil {
return err
}
debug.Log("sftp.Save", "save %v (%d bytes) to %v", h, len(p), filename)
n, err := tmpfile.Write(p)