From 713736a6a738aa371176241e11e2f7e06f63523b Mon Sep 17 00:00:00 2001 From: Nick Mathewson Date: Thu, 8 Nov 2012 23:01:39 -0500 Subject: [PATCH] Fix a memory leak in handling errors on CERTS cells. bug 7422 --- changes/bug7422 | 3 +++ src/or/channeltls.c | 2 +- 2 files changed, 4 insertions(+), 1 deletion(-) create mode 100644 changes/bug7422 diff --git a/changes/bug7422 b/changes/bug7422 new file mode 100644 index 0000000000..652f1a2231 --- /dev/null +++ b/changes/bug7422 @@ -0,0 +1,3 @@ + o Minor bugfixes: + - Fix a memory leak on failing cases of channel_tls_process_certs_cell. + Fixes bug 7422; bugfix on 0.2.4.4-alpha. diff --git a/src/or/channeltls.c b/src/or/channeltls.c index d094d15af0..ede245894e 100644 --- a/src/or/channeltls.c +++ b/src/or/channeltls.c @@ -1522,7 +1522,7 @@ channel_tls_process_certs_cell(var_cell_t *cell, channel_tls_t *chan) safe_str(chan->conn->base_.address), \ chan->conn->base_.port, (s)); \ connection_or_close_for_error(chan->conn, 0); \ - return; \ + goto err; \ } while (0) if (chan->conn->base_.state != OR_CONN_STATE_OR_HANDSHAKING_V3)