- John Wilkinson filed bug #2155496

  (http://curl.haxx.se/bug/view.cgi?id=2155496) pointing out an error case
  without a proper human-readable error message. When a read callback returns
  a too large value (like when trying to return a negative number) it would
  trigger and the generic error message then makes the proplem slightly
  different to track down. I've added an error message for this now.

diff --git a/CHANGES b/CHANGES
index e7553768912678de7a2cd9c088e71f8a1d3f16b6..22eee97219a36b1c21b7d6142a07e6d381837a71 100644 (file)
--- a/CHANGES
+++ b/CHANGES
@@ -6,6 +6,14 @@
 
                                   Changelog
 
+Daniel Stenberg (15 Oct 2008)
+- John Wilkinson filed bug #2155496
+  (http://curl.haxx.se/bug/view.cgi?id=2155496) pointing out an error case
+  without a proper human-readable error message. When a read callback returns
+  a too large value (like when trying to return a negative number) it would
+  trigger and the generic error message then makes the proplem slightly
+  different to track down. I've added an error message for this now.
+
 Daniel Fandrich (9 Oct 2008)
 - Fixed the --interface option to work with IPv6 connections on glibc
   systems supporting getifaddrs(). Also fixed a problem where an IPv6

diff --git a/lib/transfer.c b/lib/transfer.c
index bd6629916b622ae358614ad508f3563222fb5ea0..f15d9a0f25e44429770ed13809ddae69b529363c 100644 (file)
--- a/lib/transfer.c
+++ b/lib/transfer.c
@@ -152,6 +152,7 @@ CURLcode Curl_fillreadbuffer(struct connectdata *conn, int bytes, int *nreadp)
   else if((size_t)nread > buffersize) {
     /* the read function returned a too large value */
     *nreadp = 0;
+    failf(data, "read function returned funny value");
     return CURLE_READ_ERROR;
   }