summaryrefslogtreecommitdiff
path: root/paramiko/sftp_file.py
diff options
context:
space:
mode:
authorStephen C. Pope <stephen.pope@predict.com>2015-08-23 09:30:21 -0600
committerJeff Forcier <jeff@bitprophet.org>2016-04-24 21:45:31 -0700
commit9e76f38045ef1e24eb2f6ff93d0b68470b237bad (patch)
tree43b4bcfb63d9fd889d694e8eba17ea4ba7352aa1 /paramiko/sftp_file.py
parentd29a1fc52535aafc3aa23c58e9060c44af33d99b (diff)
downloadparamiko-9e76f38045ef1e24eb2f6ff93d0b68470b237bad.tar.gz
fix sftp stalls
Diffstat (limited to 'paramiko/sftp_file.py')
-rw-r--r--paramiko/sftp_file.py20
1 files changed, 12 insertions, 8 deletions
diff --git a/paramiko/sftp_file.py b/paramiko/sftp_file.py
index d0a37da3..a77bc926 100644
--- a/paramiko/sftp_file.py
+++ b/paramiko/sftp_file.py
@@ -460,8 +460,8 @@ class SFTPFile (BufferedFile):
# do these read requests in a temporary thread because there may be
# a lot of them, so it may block.
for offset, length in chunks:
+ num = self.sftp._async_request(self, CMD_READ, self.handle, long(offset), int(length))
with self._prefetch_lock:
- num = self.sftp._async_request(self, CMD_READ, self.handle, long(offset), int(length))
self._prefetch_extents[num] = (offset, length)
def _async_response(self, t, msg, num):
@@ -475,13 +475,17 @@ class SFTPFile (BufferedFile):
if t != CMD_DATA:
raise SFTPError('Expected data')
data = msg.get_string()
- with self._prefetch_lock:
- offset, length = self._prefetch_extents[num]
- self._prefetch_data[offset] = data
- del self._prefetch_extents[num]
- if len(self._prefetch_extents) == 0:
- self._prefetch_done = True
-
+ while True:
+ with self._prefetch_lock:
+ # spin if in race with _prefetch_thread
+ if self._prefetch_extents.has_key(num):
+ offset, length = self._prefetch_extents[num]
+ self._prefetch_data[offset] = data
+ del self._prefetch_extents[num]
+ if len(self._prefetch_extents) == 0:
+ self._prefetch_done = True
+ break
+
def _check_exception(self):
"""if there's a saved exception, raise & clear it"""
if self._saved_exception is not None: