From 8d8afc4efa538682ef2b80f6664456b34b84e519 Mon Sep 17 00:00:00 2001 From: David Goulet Date: Fri, 22 Jul 2022 13:53:52 -0400 Subject: relay: Send DESTROY cell instead of TRUNCATED cell Note that with this commit, TRUNCATED cells won't be used anymore that is client and relays won't emit them. Fixes #40623 Signed-off-by: David Goulet --- changes/ticket40623 | 4 ++++ src/core/or/command.c | 10 +++++----- 2 files changed, 9 insertions(+), 5 deletions(-) create mode 100644 changes/ticket40623 diff --git a/changes/ticket40623 b/changes/ticket40623 new file mode 100644 index 0000000000..d2a0e7eaad --- /dev/null +++ b/changes/ticket40623 @@ -0,0 +1,4 @@ + o Major bugfixes (relay): + - Stop sending TRUNCATED cell and instead close the circuits which sends a + DESTROY cell so every relay in the circuit path can stop queuing cells. + Fixes bug 40623; bugfix on 0.1.0.2-rc. diff --git a/src/core/or/command.c b/src/core/or/command.c index 9226309ff7..6ddfc317d6 100644 --- a/src/core/or/command.c +++ b/src/core/or/command.c @@ -637,11 +637,11 @@ command_process_destroy_cell(cell_t *cell, channel_t *chan) if (CIRCUIT_IS_ORIGIN(circ)) { circuit_mark_for_close(circ, reason|END_CIRC_REASON_FLAG_REMOTE); } else { - char payload[1]; - log_debug(LD_OR, "Delivering 'truncated' back."); - payload[0] = (char)reason; - relay_send_command_from_edge(0, circ, RELAY_COMMAND_TRUNCATED, - payload, sizeof(payload), NULL); + /* Close the circuit so we stop queuing cells for it and propagate the + * DESTROY cell down the circuit so relays can stop queuing in-flight + * cells for this circuit which helps with memory pressure. */ + log_debug(LD_OR, "Received DESTROY cell from n_chan, closing circuit."); + circuit_mark_for_close(circ, END_CIRC_REASON_TORPROTOCOL); } } } -- cgit v1.2.3-54-g00ecf