always do "forced" window updates
When calling _libssh2_channel_receive_window_adjust() internally, we now
always use the 'force' option to prevent libssh2 to avoid sending the
update if the update isn't big enough.
It isn't fully analyzed but we have seen corner cases which made a
necessary window update not get send due to this and then the other side
doesn't send data our side then sits waiting for forever.
diff --git a/src/channel.c b/src/channel.c
index 5181f6f..63e5d5b 100644
--- a/src/channel.c
+++ b/src/channel.c
@@ -1418,7 +1418,7 @@
rc = _libssh2_channel_receive_window_adjust(channel,
channel->flush_refund_bytes,
- 0, NULL);
+ 1, NULL);
if (rc == LIBSSH2_ERROR_EAGAIN)
return rc;
}
diff --git a/src/packet.c b/src/packet.c
index 5852d0c..364b434 100644
--- a/src/packet.c
+++ b/src/packet.c
@@ -661,7 +661,7 @@
rc = _libssh2_channel_receive_window_adjust(session->
packAdd_channelp,
datalen - 13,
- 0, NULL);
+ 1, NULL);
if (rc == LIBSSH2_ERROR_EAGAIN)
return rc;
diff --git a/src/sftp.c b/src/sftp.c
index 6d73751..96e00aa 100644
--- a/src/sftp.c
+++ b/src/sftp.c
@@ -285,7 +285,7 @@
/* ask for twice the data amount we need at once */
rc = _libssh2_channel_receive_window_adjust(channel,
sftp->partial_len*2,
- 0, NULL);
+ 1, NULL);
/* store the state so that we continue with the correct
operation at next invoke */
sftp->packet_state = (rc == LIBSSH2_ERROR_EAGAIN)?
@@ -1262,7 +1262,7 @@
rc = _libssh2_channel_receive_window_adjust(sftp->channel,
max_read_ahead*8,
- 0, NULL);
+ 1, NULL);
/* if this returns EAGAIN, we will get back to this function
at next call */
assert(rc != LIBSSH2_ERROR_EAGAIN || !filep->data_left);
