summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorNick Mathewson <nickm@torproject.org>2011-08-24 17:30:20 -0400
committerNick Mathewson <nickm@torproject.org>2011-08-24 17:31:37 -0400
commitdfcd3d9ce00dedcf00124c9329985c5522f286dc (patch)
tree2b8e5ef9064ad4340708f974d4c9ee2052201be3
parentf186e16241f7d6a0090260f87ce412a0fb7ec47a (diff)
downloadtor-dfcd3d9ce00dedcf00124c9329985c5522f286dc.tar.gz
tor-dfcd3d9ce00dedcf00124c9329985c5522f286dc.zip
Set write low-watermarks on all bufferevents.
If we don't do this, then we never invoke the bufferevent write callbacks until all the bufferevent's data is flushed.
-rw-r--r--changes/bug38055
-rw-r--r--src/or/connection.c5
2 files changed, 10 insertions, 0 deletions
diff --git a/changes/bug3805 b/changes/bug3805
new file mode 100644
index 0000000000..9d12b8146e
--- /dev/null
+++ b/changes/bug3805
@@ -0,0 +1,5 @@
+ o Major bugfixes (bufferevents):
+ - Correctly notice when data has been written from a bufferevent
+ without flushing it completely. Bugfix on 0.2.3.1-alpha; fixes
+ bug 3805.
+
diff --git a/src/or/connection.c b/src/or/connection.c
index c4cbef4c9d..dc804ddcec 100644
--- a/src/or/connection.c
+++ b/src/or/connection.c
@@ -2947,6 +2947,11 @@ connection_configure_bufferevent_callbacks(connection_t *conn)
connection_handle_write_cb,
connection_handle_event_cb,
conn);
+ /* Set a fairly high write low-watermark so that we get the write callback
+ called whenever data is written to bring us under 128K. Leave the
+ high-watermark at 0.
+ */
+ bufferevent_setwatermark(bufev, EV_WRITE, 128*1024, 0);
input = bufferevent_get_input(bufev);
output = bufferevent_get_output(bufev);