diff options
author | Lars Wirzenius <lars.wirzenius@codethink.co.uk> | 2014-09-30 10:17:05 +0100 |
---|---|---|
committer | Pedro Alvarez <pedro.alvarez@codethink.co.uk> | 2014-10-02 14:48:28 +0100 |
commit | ac0c001c814d25c09d9a60b2fa6fc76e3f611181 (patch) | |
tree | b4aeb64aad86318ea77ba7a36a550bbb3c9c845f | |
parent | 963dca9868938241740d63ec1475c5f887513e0c (diff) | |
download | morph-ac0c001c814d25c09d9a60b2fa6fc76e3f611181.tar.gz |
xfer-hole: Fix bug in copy_slice_from_file.
os.read is limited to an int in size. copy_slice_from_file was trying
to os.read more than that causing an OverflowError.
-rwxr-xr-x | morphlib/xfer-hole | 10 |
1 files changed, 8 insertions, 2 deletions
diff --git a/morphlib/xfer-hole b/morphlib/xfer-hole index 0d4cee7a..22ee06bf 100755 --- a/morphlib/xfer-hole +++ b/morphlib/xfer-hole @@ -120,8 +120,14 @@ def make_xfer_instructions(fd): def copy_slice_from_file(to, fd, start, end): safe_lseek(fd, start, os.SEEK_SET) - data = os.read(fd, end - start) - to.write(data) + nbytes = end - start + max_at_a_time = 1024**2 + while nbytes > 0: + data = os.read(fd, min(nbytes, max_at_a_time)) + if not data: + break + to.write(data) + nbytes -= len(data) for kind, start, end in make_xfer_instructions(fd): |