Commented Issue: Problem with sftp uploading [1539]

When I try to upload in loop several huge files (17 files on 2 gb) I see that 3 files have uploaded fully, then processing have hunged one hour and I get exception


System.Net.Sockets.SocketException (0x80004005): An existing connection
was forcibly closed by the remote host at Renci.SshNet.Sftp.SubsystemSession.WaitHandle(WaitHandle
waitHandle, TimeSpan operationTimeout) at Renci.SshNet.SftpClient.InternalUploadFile(Stream input, String path, Flags flags, SftpUploadAsyncResult asyncResult, Action`1 uploadCallback)

I think that there is problem is in this part of code in methods Renci.SshNet.SftpClient.InternalUploadFile

do{ // Cancel upload
if (asyncResult != null && asyncResult.IsUploadCanceled)
break;

if (bytesRead > 0)
{
if (bytesRead < this.BufferSize)
{
// Replace buffer for last chunk of data
var data = new byte[bytesRead];
Buffer.BlockCopy(buffer, 0, data, 0, bytesRead);
buffer = data;
}

var writtenBytes = offset + (ulong)buffer.Length;
this._sftpSession.RequestWrite(handle, offset, buffer, null, (s) =>
{
if (s.StatusCode == StatusCodes.Ok)
{
expectedResponses--;
responseReceivedWaitHandle.Set();

// Call callback to report number of bytes written
if (uploadCallback != null)
{
// Execute callback on different thread
this.ExecuteThread(() => { uploadCallback(writtenBytes); });
}
}
});
expectedResponses++;

offset += (uint)bytesRead;

bytesRead = input.Read(buffer, 0, buffer.Length);
}
else if (expectedResponses > 0)
{
// Wait for expectedResponses to change
this._sftpSession.WaitHandle(responseReceivedWaitHandle, this.OperationTimeout);
}
} while (expectedResponses > 0 || bytesRead > 0);

this._sftpSession.RequestClose(handle);
}

I try to change expectedResponses-- and expectedResponses++ on
Interlocked.Decrement(ref expectedResponses); Interlocked.Increment(ref expectedResponses);
I hope that it is solve but i'm not sure. I can test it only on Monday
Comments: ** Comment from web user: olegkap **

Hi,

Thanks for letting me know.

I just committed code with your proposed fix.

Thanks,
Oleg