chunked-encoding with Content-Length: header problem

Akos Pasztory filed debian bug report #572276
http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=572276
mentioning a problem with a resource that returns chunked-encoded
_and_ with a Content-Length and libcurl failed to properly ignore
the latter information.

diff --git a/CHANGES b/CHANGES
index 174fbc69f55b893aa3bc0294bc01606be6d64bd3..5fe6337b8e6f27f46e5f871dbac485b4d8b3a109 100644 (file)
--- a/CHANGES
+++ b/CHANGES
@@ -7,6 +7,11 @@
                                   Changelog
 
 Daniel Stenberg (23 Mar 2010)
+- Akos Pasztory filed debian bug report #572276
+  http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=572276 mentioning a problem
+  with a resource that returns chunked-encoded _and_ with a Content-Length
+  and libcurl failed to properly ignore the latter information.
+
 - Hauke Duden provided an example program that made the multi interface crash.
   His example simply used the multi interface and did first one FTP transfer
   and after completion it used a second easy handle and did another FTP

diff --git a/RELEASE-NOTES b/RELEASE-NOTES
index 6d0b7963dcb8470df48bc99172ac97a6ce9a2739..c682b9cb784fad13435f6fe810be1bfb21027b79 100644 (file)
--- a/RELEASE-NOTES
+++ b/RELEASE-NOTES
@@ -37,6 +37,7 @@ This release includes the following bugfixes:
  o SSL possible double free when reusing curl handle
  o alarm()-based DNS timeout bug
  o re-used FTP connection multi interface crash
+ o chunked-encoding with Content-Length: header problem
 
 This release includes the following known bugs:
 
@@ -48,6 +49,7 @@ advice from friends like these:
  Steven M. Schweda, Yang Tse, Jack Zhang, Tom Donovan, Martin Hager,
  Daniel Fandrich, Patrick Monnerat, Pat Ray, Wesley Miaw, Ben Greear,
  Ryan Chan, Markus Duft, Andrei Benea, Jacob Moshenko, Daniel Johnson,
- Constantine Sapuntzakis, Douglas Steinwand, Thomas Lopatic, Hauke Duden
+ Constantine Sapuntzakis, Douglas Steinwand, Thomas Lopatic, Hauke Duden,
+ Akos Pasztory
 
         Thanks! (and sorry if I forgot to mention someone)

diff --git a/lib/http.c b/lib/http.c
index b6c01395ab2187ebe4260e4363c0211880b52ac6..d4b99fd589517d338bdce35b3acc322a82183467 100644 (file)
--- a/lib/http.c
+++ b/lib/http.c
@@ -3338,8 +3338,7 @@ CURLcode Curl_http_readwrite_headers(struct SessionHandle *data,
              using chunked Transfer-Encoding.
           */
           if(k->chunk)
-            k->size=-1;
-
+            k->maxdownload = k->size = -1;
         }
         if(-1 != k->size) {
           /* We do this operation even if no_body is true, since this