check: fix successful retries with --download counting errors

See: https://forum.rclone.org/t/tons-of-data-corruption-after-rclone-copy-to-mega-can-rclone-correct-it/17017
This commit is contained in:
Nick Craig-Wood 2020-06-20 14:50:40 +01:00
parent 6606602f1e
commit ce394426b0

View File

@ -1004,7 +1004,7 @@ func checkIdenticalDownload(ctx context.Context, dst, src fs.Object) (differ boo
} }
tr1 := accounting.Stats(ctx).NewTransfer(dst) tr1 := accounting.Stats(ctx).NewTransfer(dst)
defer func() { defer func() {
tr1.Done(err) tr1.Done(nil) // error handling is done by the caller
}() }()
in1 = tr1.Account(in1).WithBuffer() // account and buffer the transfer in1 = tr1.Account(in1).WithBuffer() // account and buffer the transfer
@ -1014,7 +1014,7 @@ func checkIdenticalDownload(ctx context.Context, dst, src fs.Object) (differ boo
} }
tr2 := accounting.Stats(ctx).NewTransfer(dst) tr2 := accounting.Stats(ctx).NewTransfer(dst)
defer func() { defer func() {
tr2.Done(err) tr2.Done(nil) // error handling is done by the caller
}() }()
in2 = tr2.Account(in2).WithBuffer() // account and buffer the transfer in2 = tr2.Account(in2).WithBuffer() // account and buffer the transfer