diff options
| author | Garrett D'Amore <garrett@damore.org> | 2019-12-29 10:12:23 -0800 |
|---|---|---|
| committer | Garrett D'Amore <garrett@damore.org> | 2019-12-29 10:12:23 -0800 |
| commit | 1a9b1a0ad3e1e98a307c8a99211ebc3a94b4ac3e (patch) | |
| tree | 247a13ce5bdbbba8bff80954b76057dca8eb4c24 /src/platform | |
| parent | aed9697d71798d923443f3fb6297cecf89bc4218 (diff) | |
| download | nng-1a9b1a0ad3e1e98a307c8a99211ebc3a94b4ac3e.tar.gz nng-1a9b1a0ad3e1e98a307c8a99211ebc3a94b4ac3e.tar.bz2 nng-1a9b1a0ad3e1e98a307c8a99211ebc3a94b4ac3e.zip | |
fixes #1065 resolver leaks work structures
This includes changes to support setting the sanitizer *correctly*
(the old code CMake stuff didn't quite get it right), and addresses
a number of failures in the test code found by the address sanitizer.
Diffstat (limited to 'src/platform')
| -rw-r--r-- | src/platform/posix/posix_resolv_gai.c | 3 | ||||
| -rw-r--r-- | src/platform/windows/win_resolv.c | 3 |
2 files changed, 2 insertions, 4 deletions
diff --git a/src/platform/posix/posix_resolv_gai.c b/src/platform/posix/posix_resolv_gai.c index 9e577403..aa70a916 100644 --- a/src/platform/posix/posix_resolv_gai.c +++ b/src/platform/posix/posix_resolv_gai.c @@ -337,9 +337,8 @@ resolv_worker(void *notused) nni_aio_set_sockaddr(aio, &item->sa); nni_aio_finish(aio, rv, 0); - - NNI_FREE_STRUCT(item); } + NNI_FREE_STRUCT(item); } nni_mtx_unlock(&resolv_mtx); } diff --git a/src/platform/windows/win_resolv.c b/src/platform/windows/win_resolv.c index 53a39118..745f03d1 100644 --- a/src/platform/windows/win_resolv.c +++ b/src/platform/windows/win_resolv.c @@ -300,9 +300,8 @@ resolv_worker(void *notused) item->aio = NULL; nni_aio_finish(aio, rv, 0); - - NNI_FREE_STRUCT(item); } + NNI_FREE_STRUCT(item); } nni_mtx_unlock(&resolv_mtx); } |