Fix a very old bug in the header buckets readline that we somehow never

noticed in our previous uses of this bucket.

Reading the header buckets as an incoming request triggered this bug
while testing the fcgi code. (Assuming the next commit)

* buckets/headers_buckets.c
  (serf_headers_readline): Calculate the kind of newline *before*
    changing the state for the next read.


git-svn-id: https://svn.apache.org/repos/asf/serf/trunk@1714781 13f79535-47bb-0310-9956-ffa450edef68

buckets/headers_buckets.c

@@ -407,12 +407,12 @@ static apr_status_t serf_headers_readline(serf_bucket_t *bucket,
         return APR_EOF;
     }
 
-    /* we consumed this chunk. advance the state. */
-    status = consume_chunk(ctx);
-
     /* the type of newline found is easy... */
     *found = (ctx->state == READ_CRLF || ctx->state == READ_TERM)
-        ? SERF_NEWLINE_CRLF : SERF_NEWLINE_NONE;
+                ? SERF_NEWLINE_CRLF : SERF_NEWLINE_NONE;
+
+    /* we consumed this chunk. advance the state. */
+    status = consume_chunk(ctx);
 
     return status;
 }