Rev3225, Handle correctly and test out of range bigfile requests

Author: shortcutme

plugins/Bigfile/BigfilePlugin.py

@@ -426,7 +426,7 @@ def __init__(self, site, inner_path, prebuffer=0):
 
     def read(self, buff=64 * 1024):
         pos = self.f.tell()
-        read_until = pos + buff
+        read_until = min(self.size, pos + buff)
         requests = []
         # Request all required blocks
         while 1:

plugins/Bigfile/Test/TestBigfile.py

@@ -209,6 +209,16 @@ def testOpenBigfile(self, file_server, site, site_temp):
 
             assert len(requests) == 2  # Two block download
 
+            # Test out of range request
+            f.seek(5 * 1024 * 1024)
+            data = f.read(1024 * 1024 * 30)
+            assert len(data) == 10 * 1000 * 1000 - (5 * 1024 * 1024)
+
+            f.seek(30 * 1024 * 1024)
+            data = f.read(1024 * 1024 * 30)
+            assert len(data) == 0
+
+
 
     @pytest.mark.parametrize("piecefield_obj", [BigfilePiecefield, BigfilePiecefieldPacked])
     def testPiecefield(self, piecefield_obj, site):

src/Config.py

@@ -10,7 +10,7 @@ class Config(object):
 
     def __init__(self, argv):
         self.version = "0.6.1"
-        self.rev = 3224
+        self.rev = 3225
         self.argv = argv
         self.action = None
         self.config_file = "zeronet.conf"