diff options
| author | Nick Mathewson <nickm@torproject.org> | 2017-10-23 15:55:48 -0400 |
|---|---|---|
| committer | Nick Mathewson <nickm@torproject.org> | 2017-10-23 15:55:48 -0400 |
| commit | f0a6ea0bfd6afaa718c6b3d521a5d195d47ba56d (patch) | |
| tree | 0c356d4e53bfc004ea69032c78edb2475c979d69 | |
| parent | af33fdd7c1860399fe8d6861c163e5d64b0292b9 (diff) | |
| parent | 9ae4ffc0763fcb0a50d0d02be7d3362cc496adc1 (diff) | |
| download | tor-f0a6ea0bfd6afaa718c6b3d521a5d195d47ba56d.tar.gz tor-f0a6ea0bfd6afaa718c6b3d521a5d195d47ba56d.zip | |
Merge branch 'maint-0.3.0' into maint-0.3.1
| -rw-r--r-- | changes/bug23874 | 3 | ||||
| -rw-r--r-- | src/or/nodelist.c | 5 |
2 files changed, 8 insertions, 0 deletions
diff --git a/changes/bug23874 b/changes/bug23874 new file mode 100644 index 0000000000..bf6620553d --- /dev/null +++ b/changes/bug23874 @@ -0,0 +1,3 @@ + o Minor bugfixes (memory safety): + - Clear the address when node_get_prim_orport() returns early. + Fixes bug 23874; bugfix on 0.2.8.2-alpha. diff --git a/src/or/nodelist.c b/src/or/nodelist.c index 3ac5c3e302..d09989d93f 100644 --- a/src/or/nodelist.c +++ b/src/or/nodelist.c @@ -1116,6 +1116,11 @@ node_get_prim_orport(const node_t *node, tor_addr_port_t *ap_out) node_assert_ok(node); tor_assert(ap_out); + /* Clear the address, as a safety precaution if calling functions ignore the + * return value */ + tor_addr_make_null(&ap_out->addr, AF_INET); + ap_out->port = 0; + /* Check ri first, because rewrite_node_address_for_bridge() updates * node->ri with the configured bridge address. */ |