aboutsummaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
Diffstat (limited to 'src')
-rw-r--r--src/or/connection.c9
1 files changed, 8 insertions, 1 deletions
diff --git a/src/or/connection.c b/src/or/connection.c
index 891c81a0e7..54edcea73d 100644
--- a/src/or/connection.c
+++ b/src/or/connection.c
@@ -1860,9 +1860,16 @@ loop_again:
/* The other side's handle_write will never actually get called, so
* we need to invoke the appropriate callbacks ourself. */
connection_t *linked = conn->linked_conn;
- /* XXXX020rc Do we need to ensure that this stuff is called even if
+ /* XXXX020 Do we need to ensure that this stuff is called even if
* conn dies in a way that causes us to return -1 earlier? -NM
* No idea. -RD */
+ /* Actually, I'm pretty sure not. The big things here are to
+ * tell the linked connection, "yes, you wrote some stuff!" so that
+ * it can succeed as a appropriate. But if this side of the link
+ * returned -1, then it couldn't process the data it got. That's
+ * fairly rare, and doesn't really count as "success" for the other
+ * side. -NM
+ */
if (n_read) {
/* Probably a no-op, but hey. */