Fix for SF bug 988120 via patch 1061941.

jeremyhylton · jeremyhylton · commit def9d2a17c6f · 2004-11-07T16:13:49.000Z
If read() returned less than the number of bytes request, the full amount was subtracted from length instead of the actually read amount.
diff --git a/Lib/httplib.py b/Lib/httplib.py
@@ -457,23 +457,25 @@ def read(self, amt=None):
 
         if amt is None:
             # unbounded read
-            if self.will_close:
+            if self.length is None:
                 s = self.fp.read()
             else:
                 s = self._safe_read(self.length)
+                self.length = 0
             self.close()        # we read everything
             return s
 
         if self.length is not None:
             if amt > self.length:
                 # clip the read to the "end of response"
                 amt = self.length
-            self.length -= amt
 
         # we do not use _safe_read() here because this may be a .will_close
         # connection, and the user is reading more bytes than will be provided
         # (for example, reading in 1k chunks)
         s = self.fp.read(amt)
+        if self.length is not None:
+            self.length -= len(s)
 
         return s
 
diff --git a/Misc/ACKS b/Misc/ACKS
@@ -302,6 +302,7 @@ Jiba
 Orjan Johansen
 Simon Johnston
 Richard Jones
+Irmen de Jong
 Lucas de Jonge
 Jens B. Jorgensen
 John Jorgensen
