scp: close transfers before sending upload errors

This change should fix the random failure in TestSCPTransferQuotaLimits
because the quota is already updated when the scp command ends.

Signed-off-by: Nicola Murino <nicola.murino@gmail.com>
This commit is contained in:
Nicola Murino
2023-12-21 18:03:07 +01:00
parent d396c24ad4
commit 61b5f97bf2
5 changed files with 37 additions and 37 deletions

View File

@@ -192,15 +192,15 @@ func (c *scpCommand) getUploadFileData(sizeToRead int64, transfer *transfer) err
for {
n, err := c.connection.channel.Read(buf)
if err != nil {
c.sendErrorMessage(transfer.Fs, err)
transfer.TransferError(err)
transfer.Close()
c.sendErrorMessage(transfer.Fs, err)
return err
}
_, err = transfer.WriteAt(buf[:n], sizeToRead-remaining)
if err != nil {
c.sendErrorMessage(transfer.Fs, err)
transfer.Close()
c.sendErrorMessage(transfer.Fs, err)
return err
}
remaining -= int64(n)