diff options
| author | Nick Mathewson <nickm@torproject.org> | 2019-07-23 12:04:59 -0400 |
|---|---|---|
| committer | Nick Mathewson <nickm@torproject.org> | 2019-08-28 09:40:53 -0400 |
| commit | 47654d32497acae23aafbe3316ae73d3c00429c8 (patch) | |
| tree | 9d9bb68efe2587b033eef4de945c4d887b2cfd76 /src/test/test_pt.c | |
| parent | 3d1f9f583a45143c1259b8926f8c57fa57fc1efb (diff) | |
| download | tor-47654d32497acae23aafbe3316ae73d3c00429c8.tar.gz tor-47654d32497acae23aafbe3316ae73d3c00429c8.zip | |
Refactor config free logic to use a single path.
The right way to free a config object is now to wrap config_free(),
always. Instead of creating an alternative free function, objects
should provide an alternative clear callback to free any fields that
the configuration manager doesn't manage.
This lets us simplify our code a little, and lets us extend the
confparse.c code to manage additional fields in config_free.
Diffstat (limited to 'src/test/test_pt.c')
| -rw-r--r-- | src/test/test_pt.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/test/test_pt.c b/src/test/test_pt.c index 87e3ba356c..362c6664db 100644 --- a/src/test/test_pt.c +++ b/src/test/test_pt.c @@ -352,7 +352,7 @@ test_pt_configure_proxy(void *arg) managed_proxy_t *mp = NULL; (void) arg; - dummy_state = tor_malloc_zero(sizeof(or_state_t)); + dummy_state = or_state_new(); MOCK(process_read_stdout, process_read_stdout_replacement); MOCK(get_or_state, |