Changeset 55ccc9a0 for lib/http_client.c

Timestamp:

2012-09-22T11:52:01Z (12 years ago)

Author:

Wilmer van der Gaast <wilmer@…>

Branches:

master

Children:

d0752e8

Parents:

daf544a

Message:

Solve a whole bunch of Twitter module crashes: Twitter responses seem to
be getting truncated sometimes. Treat this as an error in http_client
already instead of returning partial data.

File:

• lib/http_client.c (modified) (3 diffs)

lib/http_client.c

@@ -198 +198 @@
         int evil_server = 0;
         char buffer[2048];
-        char *end1, *end2;
+        char *end1, *end2, *s;
+        size_t content_length;
         int st;
         
@@ -481 +482 @@
         /* Assume that a closed connection means we're finished, this indeed
            breaks with keep-alive connections and faulty connections. */
-        req->finished = 1;
+        /* req->finished = 1; */
 
 cleanup:
@@ -488 +489 @@
         else
                 closesocket( req->fd );
+        
+        if( ( s = get_rfc822_header( req->reply_headers, "Content-Length", 0 ) ) &&
+            sscanf( s, "%zd", &content_length ) == 1 )
+        {
+                if( content_length < req->body_size )
+                {
+                        req->status_code = -1;
+                        g_free( req->status_string );
+                        req->status_string = g_strdup( "Response truncated" );
+                }
+        }
+        g_free( s );
         
         if( getenv( "BITLBEE_DEBUG" ) && req )