r16735@catbus: nickm | 2007-11-26 14:24:58 -0500

Fix test for EVBUFFER_LENGTH in evhttp_make_header().  Since appending an empty buffer to another is a no-op, this is not really a bug-fix.


svn:r554
This commit is contained in:
Nick Mathewson 2007-11-26 19:25:09 +00:00
parent ce4ee418d2
commit a7a7a19045

4
http.c
View File

@ -441,9 +441,7 @@ evhttp_make_header(struct evhttp_connection *evcon, struct evhttp_request *req)
}
evbuffer_add(evcon->output_buffer, "\r\n", 2);
/* XXX EVBUFFER_LENGTH returns an unsigned value, so this test
* is always true. What is the intent of this test? -NM */
if (1) { // || EVBUFFER_LENGTH(req->output_buffer) >= 0) {
if (EVBUFFER_LENGTH(req->output_buffer) > 0) {
/*
* For a request, we add the POST data, for a reply, this
* is the regular data.