- 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.

2 files changed, 9 insertions, 0 deletions

diff --git a/CHANGES b/CHANGES
index e75537689..22eee9721 100644
--- 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 bd6629916..f15d9a0f2 100644
--- 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;
   }